Skip to content

docs: refresh README - #12

Merged
ChiragAgg5k merged 3 commits into
mainfrom
dev
Oct 17, 2026
Merged

ChiragAgg5k merged 3 commits into
mainfrom
dev

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

Refreshes the README with the rewritten getting-started guide and corrected badges, links and install steps from appwrite/sdk-generator#2077 and appwrite/appwrite#14613.

README files only. No code, version or changelog changes, so this does not need a release.

@hansi-codes

hansi-codes Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The updated OAuth guidance addresses the previously reported documentation issues, and no actionable defects remain in these changes.

Refreshes the Unity SDK README with updated installation and getting-started instructions, AppwriteManager and direct-client examples, TablesDB serialization guidance, and current badges and links. It also documents OAuth setup across Android, iOS, and WebGL.

Latest changes: The newest commit replaces the Android AppwriteConfig callback-registration claim with an explicit manifest intent-filter setup and clarifies that same-tab WebGL OAuth is unsupported.

Verdict New comments Fixed Still open
✅ Approved 0 2 0
📂 Walkthrough · 1
File Change
README.md Updates Android OAuth setup to require a project-specific manifest intent filter, links to that guidance, and explains that OAuth is unsupported on WebGL.
✅ Fixed since the last review · 2
  • Android OAuth callback is not registered from AppwriteConfig · README.md:48
  • WebGL OAuth navigation cannot resume the waiting SDK call · README.md:164

Reviewed the commits since 9d9bb2b · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Tier A · Looks good to merge. Summary

Comment thread README.md Outdated
Comment thread README.md Outdated

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k
ChiragAgg5k merged commit 6a1f1da into main Oct 17, 2026
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.

1 participant