-
Notifications
You must be signed in to change notification settings - Fork 58
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
📚 Documentation: Mistake in code snippets in getting started guide #62
Comments
Hey there, Thanks for reaching out. We appreciate you bringing this to our attention. Rest assured, we're aware of the issue, and we'll make sure to update it with the correct syntax. In the meantime, we recommend checking out our documentation, as it has been updated with the accurate examples you need. You can find them here: https://appwrite.io/docs/getting-started-for-server#makeRequest If you have any further questions or need any assistance, feel free to ask |
It seems like this is the file that needs to be updated to fix this. |
Hi @Haimantika, I would like to work on this issue, if it is up for grabs. |
Hi @aniketh-varma have assigned the issue to you! Thank you for showing interest in contributing to Appwrite! Happy Hacktoberfest 🎃 Notes: Please update us with your progress every 3 days, so that we know that you are working on it. |
Hi @Haimantika , if the previous contributor is unresponsive, I would like to work on this issue then please assign it to me later on. |
Hi @Haimantika, just to confirm, if this is the file to be updated. Then the PR I would create would be for the |
Hi @Haimantika, there is NO update/changes seen from previous assignee in this issue since 4-5 days? |
Yes! @aniketh-varma we are reassigning the issue to the next person due to inactivity from your end. @GuptaPratik02 the issue is yours now |
Thanks @Haimantika, will update you by today! |
hi @Haimantika , @stnguyen90 you can assign it to me incase @GuptaPratik02 is not able to finish it. |
Fixed the bug mentioned in the issue by updating README.md file - appwrite#62
Fixed the issue mentioned in sdk-for-python repository Issue - 62. Issue link - appwrite/sdk-for-python#62
Hi @Haimantika,
Kindly check and review my PR. |
@Haimantika I was waiting for your reply on the query I had asked, the confusion being an issue for the |
With that much of experience, you should have read the appwrite rules on contributing during hacktoberfest brother... they clearly stated 3 days of inactivity could roll over issue to others. Atleast you could've opened a draft PR if it was taking time |
Thanks for the advice man. I appreciate Appwrite for abiding by the rules, but a simple confirmation before re-assigning the issue would be more courteous to the current assignee and would confirm the inactivity on the issue. |
Hi @Haimantika, can you acknowledge once that pull request I made is assigned to a maintainer to review? |
Hi @aniketh-varma apologies if you feel this way, that wasn't the intention at all. We wanted to give a fair chance to everyone to contribute. Since in the past we have had cases where people didn't work on an issue and others didn't get a chance to work on it either. |
@Haimantika I would like to work on this. Can you assign it to me? |
Hey @Haimantika, I completely understand, that tracking the repos' issues, that too during the Hacktoberfest is a tedious task, and I really appreciate that Appwrite is abiding by the rules. I was just not happy about the reassignment of the issue without any confirmation from my end. There is no need for an apology, you are doing your work, I would just want to suggest, maybe a comment confirming the activity on an inactive issue may be a better alternative than just re-assigning. Thanks! 😄 |
💭 Description
The code snippet for creating a user in the getting started guide throws an error due to an invalide phone number.
This:
should be changed to:
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: