-
-
Notifications
You must be signed in to change notification settings - Fork 193
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve bug report and feature request templates #671
Conversation
I just now realized, that you also mentioned a feature request form, in #670. Let me know if you would like me to produce another copy for that as well. |
@smultar wonderful work 👍 I think for the embla version field we should make it a text input rather than a dropdown. So that it can be timeless and we don't have to update it to keep up with the current version because most likely we'll forget about it. (unless there's a standardized checklist of things to do upon every new release/version) |
@davidjerleke would it be better if I think that'd make it easier to find. unless there's a good reason why it should stay inside |
Did a little search and it seems to be recommended to place @davidjerleke if you approve and want to delegate it to me I can make a PR for relocating it. |
@zaaakher I think all three things you've mentioned are good input. Thank you for being so dedicated to help out and improve Embla.
Yes that would be great thanks! I also agree with this.
I agree. Nicely done. Thanks!
Yes let's make it a text field to avoid manual labour! Thank you both for all the help! |
@davidjerleke I'll make it an input field, give me a moment to make those changes. |
If you don't mind, and don't feel obligated to, I would appreciate if you could just skim both templates and see if you find anything strange. That would speed up the process of merging this. But I will do it eventually if you don't have time. Regardless, thanks a lot for your efforts ⭐! Best, |
Additionally, I recommend creating a
Similar to this Which will show an additional option upon issue creation like this |
This is definitely the way to go. Thank you for this valuable input 🤩. |
I just woke up in my time zone. It's -22 degrees outside, send help, please. In other news, it was a pleasure, I see that you closed this out, is there anything else I need to do? |
Hi @smultar ❄️, No this has been merged. Thanks for your contribution! Feel free to contribute again 👍. |
Created a different variant of the bug report form, that supports a checklist and requirements before submitting. Should resolve #670 and make bug reporting a bit clearer.