Skip to content

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

Merged
merged 25 commits into from
May 25, 2023

Conversation

mbfakourii
Copy link
Member

@mbfakourii mbfakourii commented May 14, 2023

Add Flutter document

Closes: #882

The following tasks must be completed

  • Getting Started
  • Objects
  • Custom Objects
  • Add New Values to Objects
  • Save Objects using Pins
  • Storage
  • Increment Counter Values
  • Array Operator in Objects
  • Queries
  • Complex Queries
  • Relational queries
  • Counting objects
  • LiveQuery
  • ParseLiveList
  • Users
  • Facebook, OAuth and 3rd Party Login/User
  • Security for Objects - ParseACL
  • Config
  • Cloud Functions
  • Relation
  • File
  • Other Features

add objects
add queries
add users
update objects.md
add other-features.md
update queries.md
add storage.md
update users.md
@mbfakourii mbfakourii changed the title feat: add flutter document feat: Add flutter document May 14, 2023
@mbfakourii
Copy link
Member Author

@parse-community/docs-review
If possible, review

Copy link
Member

@mtrezza mtrezza left a 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?

image

@mbfakourii
Copy link
Member Author

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?

image

added

Copy link
Member

@mtrezza mtrezza left a 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.

mbfakourii and others added 10 commits May 20, 2023 19:34
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>
@mbfakourii
Copy link
Member Author

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.

@mtrezza
Thanks for checking

@mtrezza
Copy link
Member

mtrezza commented May 20, 2023

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.

@mbfakourii
Copy link
Member Author

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

@mbfakourii
Copy link
Member Author

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.

I checked and fixed the problems.

Copy link
Member

@mtrezza mtrezza left a 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.

Copy link
Member

@mtrezza mtrezza left a 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?

@mbfakourii
Copy link
Member Author

Looks good! Amazing job on this huge PR, can we merge?

Yes, I think it is ready for merge

@mtrezza mtrezza changed the title feat: Add flutter document feat: Add Parse Flutter SDK documentation May 25, 2023
@mtrezza mtrezza merged commit 5465970 into parse-community:gh-pages May 25, 2023
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.

Document improvement
2 participants