-
Notifications
You must be signed in to change notification settings - Fork 66
Upgraded issue templates to use GitHub issue forms #35
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
Upgraded issue templates to use GitHub issue forms #35
Conversation
| - label: "I checked and didn't find similar issue" | ||
| required: true | ||
| - type: checkboxes | ||
| id: read-code-of-conduct |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add this code of conduct option to all the forms.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, please check
|
@aniketh-varma Great work. One more change before we can approve. We have made few changes in the language and also the format of Code of conduct. Please check https://github.com/appwrite/appwrite template and update accordingly. |
Done, please check. |
lohanidamodar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @aniketh-varma. Just a small fix.
.github/ISSUE_TEMPLATE/bug.yaml
Outdated
| - type: dropdown | ||
| id: appwrite-python-sdk-version | ||
| attributes: | ||
| label: "🎲 Appwrite Python SDK version" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep the Appwrite version here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, just the Appwrite versions have to be mentioned?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add appwrite version field.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, please take another look.
|
THANK YOU so much! All changes merged 🥳 |
Tasks Completed:
Closes #33