Skip to content
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

Upgrade Issue Templates #4

Merged
merged 6 commits into from
Oct 6, 2021
Merged

Conversation

vivekkj123
Copy link
Contributor

Hi,
I've made the changes for issue templates. If any changes needed, let me know. This PR can close #3.

Thanks,

Vivek K J
vivekkj.com

@vivekkj123
Copy link
Contributor Author

Also, please tag this repo with hacktoberfest tag, or my contributions won't counted..

@lohanidamodar
Copy link
Member

@vivekkj123 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.
Thank you.

@vivekkj123
Copy link
Contributor Author

@lohanidamodar I've updated issue templates as you said.

Copy link
Member

@lohanidamodar lohanidamodar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vivekkj123 Thank you for your hard work and patience. Few more changes to make it even better.

Comment on lines 36 to 39
attributes:
label: "🎲 Appwrite Go SDK Version"
description: "What version of Appwrite are you running?"
placeholder: "1.0.0"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We want to keep appwrite's version for consistency.

@@ -0,0 +1,33 @@
name: "📚 Documentation"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This template has some error, GitHub cannot parse. Please validate and update.

@vivekkj123
Copy link
Contributor Author

@lohanidamodar fixed them

@christyjacob4 christyjacob4 merged commit afa2094 into appwrite:master Oct 6, 2021
@christyjacob4
Copy link
Member

THANK YOU so much! All changes merged 🥳
Please reach out to me on our Discord server if you would like to claim your Appwrite swag!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade our issue templates to use GitHub issue forms ✍️
3 participants