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

Update GETTING_STARTED.md #6826

Conversation

GuptaPratik02
Copy link
Contributor

@GuptaPratik02 GuptaPratik02 commented Oct 6, 2023

Improved the doc.
Fixed the issue mentioned in sdk-for-python repository Issue-62.

As mentioned in issue, updated the Getting_Started Guide with necessary changes.
Kindly review and merge the same.
Thank you.

My Discord ID : pratikgupta9997

Fixed the issue mentioned in sdk-for-python repository Issue - 62.
Issue link - appwrite/sdk-for-python#62
Copy link
Contributor

@Haimantika Haimantika left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -23,7 +23,7 @@ Once your SDK object is set, create any of the Appwrite service objects and choo
```python
users = Users(client)

result = users.create('[USER_ID]', 'email@example.com', 'password')
result = users.create(ID.unique(), email = 'email1@example.com', phone = '+123456789', password = 'password', name = 'user1')
Copy link
Contributor

Choose a reason for hiding this comment

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

Why email1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually, that I used for demonstrating an example above.
So shall I rename it to email or it's fine?

@@ -43,7 +43,7 @@ client = Client()

users = Users(client)

result = users.create(ID.unique(), 'email@example.com', 'password')
result = users.create(ID.unique(), email = 'email1@example.com', phone = '+123456789', password = 'password', name = 'user1')
Copy link
Contributor

Choose a reason for hiding this comment

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

Why don't we use something like Walter O'Brien that's actually a name instead of user1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, that's good idea. Can use Walter O'Brien as name instead of user1, look more fancy and real.
Will make the changes and do a commit again.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @gewenyu99, committed the changes. Checkout once
Thank you.

Changed user1 name and (email1 -> email).
@GuptaPratik02
Copy link
Contributor Author

Hi @stnguyen90, the reviewers have approved the changes I made.
Kindly checkout the same.

@gewenyu99
Copy link
Contributor

@GuptaPratik02 Please wait for this to be merged by a team member :)

@gewenyu99 gewenyu99 added the hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later label Oct 19, 2023
@GuptaPratik02
Copy link
Contributor Author

GuptaPratik02 commented Oct 19, 2023

Hi @Haimantika,
As you said in Discord chat, should drop our Discord ID in PR.
My Discord ID - pratikgupta9997

"email@example.com",
"password"
"email@example.com",
"+123456789",
Copy link
Member

Choose a reason for hiding this comment

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

This method does not take a phone number

"email@example.com",
"password"
"email@example.com",
"+123456789",
Copy link
Member

Choose a reason for hiding this comment

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

This method does not take a phone number

@@ -82,7 +86,7 @@ The Appwrite Android SDK raises an `AppwriteException` object with `message`, `c

```kotlin
try {
var user = account.create(ID.unique(), "email@example.com", "password")
var user = account.create(ID.unique(),"email@example.com","+123456789","password","Walter O'Brien")
Copy link
Member

Choose a reason for hiding this comment

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

same here

password: "password"
userId: ID.unique(),
email: "email@example.com",
phone: "+123456789",
Copy link
Member

Choose a reason for hiding this comment

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

same here

password: "password"
userId: ID.unique(),
email: "email@example.com",
phone: "+123456789",
Copy link
Member

Choose a reason for hiding this comment

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

same here

Copy link
Member

@christyjacob4 christyjacob4 left a comment

Choose a reason for hiding this comment

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

Please fix the PR. The Account.create() function does not accept a phone number

@gewenyu99
Copy link
Contributor

Please fix the PR. The Account.create() function does not accept a phone number

Does it not? 👀 Huuuuhhhhhh

removed phone number from account.create() function as it does not accept it.
Removed phone number from account.create() function as it does not accept it.
Removed phone number from account.create() function as it does not accept it.
Removed phone number from account.create() function as it does not accept it.
Removed phone number from account.create() function as it does not accept it.
Removed phone number from account.create() function as it does not accept it.
@GuptaPratik02
Copy link
Contributor Author

Please fix the PR. The Account.create() function does not accept a phone number

Hi @christyjacob4, I rectified the account.create() functions as said, please review the same :)

@GuptaPratik02
Copy link
Contributor Author

GuptaPratik02 commented Oct 28, 2023

Please fix the PR. The Account.create() function does not accept a phone number

Hi @christyjacob4, I rectified the account.create() functions as said, please review the same :)

Hi @christyjacob4, I made the changes 5-6 days back earlier as said.
Kindly have a look.

My Discord ID - pratikgupta9997

@GuptaPratik02
Copy link
Contributor Author

Hi @gewenyu99, do I need to make any changes as of now?
I am not able to see the comment you posted just recently.

@gewenyu99
Copy link
Contributor

Hi @gewenyu99, do I need to make any changes as of now? I am not able to see the comment you posted just recently.

Nope, I pinged Christy to see what else he wanted. I think we can merge this.

@eldadfux eldadfux merged commit 14e81c2 into appwrite:main Feb 20, 2024
20 checks passed
@gewenyu99
Copy link
Contributor

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

@GuptaPratik02
Copy link
Contributor Author

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

Hey, thanks for the update and my PR has been merged 👍
Here is my Discord ID : pratikgupta9997

@GuptaPratik02
Copy link
Contributor Author

GuptaPratik02 commented Apr 18, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants