-
-
Notifications
You must be signed in to change notification settings - Fork 514
feat: Add Parse Flutter SDK documentation #928
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
Conversation
add objects add queries add users
update objects.md add other-features.md update queries.md add storage.md update users.md
@parse-community/docs-review |
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.
I think we also need to update something on the homepage; the Flutter Guide link currently points to the README, but it should then point to these new Flutter docs; not sure whether the homepage is also in this docs repo?

added |
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.
Good work on this massive PR! I think it just needs some refining and formatting for proper markdown syntax. Instead of marking things individually, I've only marked a few examples that need refining that occur throughout the code.
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Co-authored-by: Manuel <5673677+mtrezza@users.noreply.github.com>
@mtrezza |
Sure, these were just examples, there are more cases like this in the PR, so it would be good if someone could look through the PR and correct them. |
Yes, I will definitely check it completely |
I checked and fixed the problems. |
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.
I think it would be good for someone else to look over this and clean up some style issues. It's a quite an extensive PR, so I've gotten only to the first 50% percent.
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.
Looks good! Amazing job on this huge PR, can we merge?
Yes, I think it is ready for merge |
Add Flutter document
Closes: #882
The following tasks must be completed