Skip to content

Upgraded issue templates to use GitHub issue forms#5

Merged
lohanidamodar merged 10 commits intoutopia-php:mainfrom
alceil:main
Oct 25, 2021
Merged

Upgraded issue templates to use GitHub issue forms#5
lohanidamodar merged 10 commits intoutopia-php:mainfrom
alceil:main

Conversation

@alceil
Copy link
Contributor

@alceil alceil commented Oct 1, 2021

pull request for #4

Copy link
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.

Hey, 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

@alceil
Copy link
Contributor Author

alceil commented Oct 2, 2021

sure got it @lohanidamodar

@alceil
Copy link
Contributor Author

alceil commented Oct 2, 2021

I have added the required changes @lohanidamodar

- type: dropdown
id: appwrite-websocket
attributes:
label: "🎲 Appwrite version"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
label: "🎲 Appwrite version"
label: "🎲 Utopia Websocket version"

Copy link
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.

@alceil good job. Just few more fixes.

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

Choose a reason for hiding this comment

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

Please use the same code of conduct field as the bug template.

@alceil
Copy link
Contributor Author

alceil commented Oct 4, 2021

Did the required changes can you have a look @lohanidamodar

@lohanidamodar
Copy link
Contributor

@alceil great work 🚀 Please feel free to look for more PRs you might be interested in helping with on our long list of Hacktoberfest friendly issues to help make Appwrite better.

@alceil
Copy link
Contributor Author

alceil commented Oct 5, 2021

Sure, i will @lohanidamodar

@lohanidamodar lohanidamodar merged commit 42ce920 into utopia-php:main Oct 25, 2021
@lohanidamodar
Copy link
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 :)

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.

2 participants