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

separated anonymous and magic URL by comma #184

Merged
merged 2 commits into from
Oct 5, 2023
Merged

separated anonymous and magic URL by comma #184

merged 2 commits into from
Oct 5, 2023

Conversation

PratikDev
Copy link
Contributor

separated anonymous and magic URL by comma, removed unnecessary space, and made all method names Camal Case

What does this PR do?

Fixes punctuation issues in the doc so that any readers get a better understanding of the documentation

Test Plan

No tests necessary

Related PRs and Issues

Not related to any other PR

Have you read the Contributing Guidelines on issues?

Yes

separated `anonymous` and `magic URL` by comma, removed unnecessary space, and made all method names Camal Case
@vercel
Copy link

vercel bot commented Oct 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 5, 2023 8:26am

@gewenyu99
Copy link
Contributor

These aren't proper nouns, so I wouldn't do anything with the casing.

If you could just add the comma and leave the rest alone, would be great ❤️

Copy link
Contributor

@gewenyu99 gewenyu99 left a comment

Choose a reason for hiding this comment

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

Please remove the case changes and only add a comma.

@PratikDev
Copy link
Contributor Author

did it

Copy link
Contributor

@gewenyu99 gewenyu99 left a comment

Choose a reason for hiding this comment

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

LGTM

@gewenyu99 gewenyu99 merged commit f141e3f into appwrite:main Oct 5, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants