Skip to content

Issue templates for the repo using new github issue forms#19

Merged
lohanidamodar merged 4 commits intoutopia-php:mainfrom
avats-dev:main
Oct 25, 2021
Merged

Issue templates for the repo using new github issue forms#19
lohanidamodar merged 4 commits intoutopia-php:mainfrom
avats-dev:main

Conversation

@avats-dev
Copy link
Copy Markdown
Contributor

Resolves #18

This PR adds issue templates using new interactive github issue forms feature for following tasks:

  • bug reporting
  • documentation issues
  • feature suggestions

Better ease of use for issue reporting with interactive form.

…porting, documentation issues and feature suggestions.
Copy link
Copy Markdown
Contributor

@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.

@avats-dev thank you for your PR. Great work. I have few suggestions before we can approve this.
Thank you

Comment thread .github/ISSUE_TEMPLATE/bug.yaml Outdated
Comment thread .github/ISSUE_TEMPLATE/bug.yaml Outdated
Comment thread .github/ISSUE_TEMPLATE/documentation.yaml Outdated
Comment thread .github/ISSUE_TEMPLATE/feature.yaml Outdated
Comment thread .github/ISSUE_TEMPLATE/feature.yaml Outdated
@lohanidamodar
Copy link
Copy Markdown
Contributor

Hey @avats-dev, we have few more changes required. For example you can ask for the library version, php version and code of conduct on all the templates. We have a first approved PR please take that as reference
utopia-php/http#28

@avats-dev
Copy link
Copy Markdown
Contributor Author

@lohanidamodar I have updated the PR with suitable changes as requested in suggestions. Please review it and LMK if something is missing. Happy to contribute! 🎉

Copy link
Copy Markdown
Contributor

@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.

@avats-dev great job. Just some small fixes.

placeholder: "I use Cloudflare for ..."
- type: checkboxes
id: no-duplicate-issues
attributes:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's add code of conduct field.

- label: "I checked and didn't find similar issue"
required: true
- type: checkboxes
id: read-code-of-conduct
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

let's make code of conduct consistent. Take reference from
utopia-php/http#28

description: "Please explain why this feature should be implemented and how it would be used. Add examples, if applicable."
placeholder: "In my use-case, ..."
- type: checkboxes
id: no-duplicate-issues
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's add code of conduct.
utopia-php/http#28

@avats-dev
Copy link
Copy Markdown
Contributor Author

@lohanidamodar Thanks for the review, I'll update the changes soon. 🎉

@avats-dev
Copy link
Copy Markdown
Contributor Author

I have updated the required changes, please review it and LMK if you need any more teaks. Thanks 🎉

@lohanidamodar
Copy link
Copy Markdown
Contributor

THANK YOU! All changes merged 🥳

Please reach out to eldadfux on our Discord server if you would like to claim your Appwrite swags! As a way of saying thank you, we would also love to invite you to join the Appwrite organization on GitHub. Please share your GitHub username with us on Discord.

You can accept the invite by visiting https://github.com/orgs/appwrite/invitation. By joining our team, you will officially be an Appwrite maintainer on GitHub.

You can change your membership visibility settings, so your new Appwrite team membership badge will show up on your personal GitHub profile.

Please feel free to look for more PRs you might be interested in helping with on our long list of Hacktoberfest friendly issues and help make Appwrite better :)

@lohanidamodar lohanidamodar merged commit 05d94ad into utopia-php:main Oct 25, 2021
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 ✍️

2 participants