-
Notifications
You must be signed in to change notification settings - Fork 226
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
fix: grammar mistakes #552
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, thanks for the PR, there's just one thing, we don't use :
colons before lists as a stylistic choice.
- Impose the same rules upon the organization’s sub-processors who assist in providing Appwrite’s services as described in the Terms of Service (“ToS”). | ||
- Appwrite will notify users promptly about policy changes and/or data breaches. | ||
|
||
You can learn more in our [Privacy policy](https://appwrite.io/privacy) and [Cookie policy](https://appwrite.io/cookies). | ||
You can also reach us at `privacy@appwrite.io` for more questions. | ||
|
||
Appwrite has also implemented the following security measures to achieve technical compliance. | ||
Appwrite has also implemented the following security measures to achieve technical compliance: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, you do it at line 10 of that same file... :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But can remove both if you want
Yo boys I actually fixed it so now it should be ready to get merged :) |
Thanks for your contribution 🚀 |
What does this PR do?
Ths PR fixes some grammar mistakes I saw while reading through appwrite's docs. There are more dotted around, but I just fixed the few I saw myself :)
Test Plan
Not needed.
Related PRs and Issues
None.
Have you read the Contributing Guidelines on issues?
Yes