-
Notifications
You must be signed in to change notification settings - Fork 298
Full schema #2676
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
Full schema #2676
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the WalkthroughA new blog post was added at src/routes/blog/post/announcing-full-schema-creation/+page.markdoc containing front matter and a full article describing the "Full Schema Creation" feature for Appwrite Databases. A changelog entry was added at src/routes/changelog/(entries)/2025-12-30 documenting the same feature with front matter and a summary. An author page at src/routes/blog/author/aditya-oberai/+page.markdoc was updated to change the role (Developer Advocate -> Developer Relations Lead) and to update the twitter URL. All changes are content/metadata only; no code or runtime behavior was modified. Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Pre-merge checks❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Actionable comments posted: 3
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
static/images/blog/full-schema.pngis excluded by!**/*.png
📒 Files selected for processing (2)
src/routes/blog/post/announcing-full-schema-creation/+page.markdocsrc/routes/changelog/(entries)/2025-12-29
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: build
- GitHub Check: assets
- GitHub Check: tests
🔇 Additional comments (1)
src/routes/changelog/(entries)/2025-12-29 (1)
1-14: LGTM: Changelog entry is clear and well-structured.The changelog entry effectively summarizes the Full Schema Creation feature with clear, concise language and appropriate metadata.
| layout: post | ||
| title: Announcing Full Schema Creation: Provision complete tables in one atomic call | ||
| description: Create a table, all its columns, and indexes synchronously, ready to use instantly, with no background jobs. | ||
| date: 2024-12-12 |
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.
Fix the date inconsistency.
The blog post date is set to 2024-12-12, but the corresponding changelog entry uses 2025-12-29. These dates are over a year apart, which will cause chronological ordering issues and user confusion.
Verify which date is correct and update accordingly. Most likely this should be 2025-12-12 to align with the changelog month.
🔎 Proposed fix
-date: 2024-12-12
+date: 2025-12-12📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| date: 2024-12-12 | |
| date: 2025-12-12 |
🤖 Prompt for AI Agents
In src/routes/blog/post/announcing-full-schema-creation/+page.markdoc around
line 5, the frontmatter date is 2024-12-12 but the changelog uses 2025-12-29;
update the blog post date to the correct year to match the changelog (most
likely change the date to 2025-12-12), or confirm the authoritative date and
make both the post frontmatter and the changelog consistent; ensure the format
remains YYYY-MM-DD.
src/routes/blog/post/announcing-full-schema-creation/+page.markdoc
Outdated
Show resolved
Hide resolved
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.
@LauraDuRy it would also be good to add a code example here
We can use this:
const table = await tablesDB.createTable({
databaseId: 'contacts_db',
tableId: 'contacts',
name: 'Contacts',
columns: [
{
key: 'email',
type: 'email',
required: true
},
{
key: 'name',
type: 'string',
size: 255,
required: true
},
{
key: 'is_active',
type: 'boolean',
required: true
},
],
indexes: [
{
key: 'idx_email',
type: 'unique',
attributes: ['email']
}
]
});| - [Announcing API for spatial columns: Build scalable location-aware apps with ease](https://appwrite.io/blog/post/announcing-spatial-columns) | ||
| - [Announcing an improved Appwrite Databases experience. A completely new look and feel](https://appwrite.io/blog/post/announcing-appwrite-databases-new-ui) | ||
| - [Announcing Atomic numeric operations: Safe, server-side increments and decrements](https://appwrite.io/blog/post/announcing-atomic-numeric-operations) |
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.
| - [Announcing API for spatial columns: Build scalable location-aware apps with ease](https://appwrite.io/blog/post/announcing-spatial-columns) | |
| - [Announcing an improved Appwrite Databases experience. A completely new look and feel](https://appwrite.io/blog/post/announcing-appwrite-databases-new-ui) | |
| - [Announcing Atomic numeric operations: Safe, server-side increments and decrements](https://appwrite.io/blog/post/announcing-atomic-numeric-operations) | |
| - [Announcing API for spatial columns: Build scalable location-aware apps with ease](/blog/post/announcing-spatial-columns) | |
| - [Announcing an improved Appwrite Databases experience. A completely new look and feel](/blog/post/announcing-appwrite-databases-new-ui) | |
| - [Announcing Atomic numeric operations: Safe, server-side increments and decrements](/blog/post/announcing-atomic-numeric-operations) |
| layout: post | ||
| title: Announcing Full Schema Creation: Provision complete tables in one atomic call | ||
| description: Create a table, all its columns, and indexes synchronously, ready to use instantly, with no background jobs. | ||
| date: 2024-12-12 |
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.
@LauraDuRy the date is incorrect
src/routes/blog/post/announcing-full-schema-creation/+page.markdoc
Outdated
Show resolved
Hide resolved
src/routes/blog/post/announcing-full-schema-creation/+page.markdoc
Outdated
Show resolved
Hide resolved
src/routes/blog/post/announcing-full-schema-creation/+page.markdoc
Outdated
Show resolved
Hide resolved
src/routes/blog/post/announcing-full-schema-creation/+page.markdoc
Outdated
Show resolved
Hide resolved
| @@ -0,0 +1,102 @@ | |||
| --- | |||
| layout: post | |||
| title: Announcing Full Schema Creation: Provision complete tables in one atomic call | |||
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.
| title: Announcing Full Schema Creation: Provision complete tables in one atomic call | |
| title: "Announcing Full Schema Creation: Provision complete tables in one atomic call" |
src/routes/blog/post/announcing-full-schema-creation/+page.markdoc
Outdated
Show resolved
Hide resolved
Co-authored-by: Aditya Oberai <adityaoberai1@gmail.com>
What does this PR do?
(Provide a description of what this PR does.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)
Related PRs and Issues
(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)
Have you read the Contributing Guidelines on issues?
(Write your answer here.)
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.