Skip to content

Conversation

@abnegate
Copy link
Member

@abnegate abnegate commented Jan 20, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Improved date field normalization in Appwrite migrations. Timestamp values for created and updated dates are now consistently normalized across all migrated resources, including databases, tables, columns, and indexes to ensure proper data integrity and consistency during the migration process.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 20, 2026

Walkthrough

The change modifies src/Migration/Destinations/Appwrite.php to introduce date field normalization for Appwrite migration documents. Before constructing documents in the createDatabase, createTable, createColumn, and createIndex methods, the code now computes normalized createdAt and updatedAt values using a normalizeDateTime() function. These normalized values replace the direct method calls to getCreatedAt() and getUpdatedAt() in the document payloads. The normalization pattern is applied consistently across all four methods with no changes to control flow or public interfaces.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Fix other stamps' is vague and does not clearly convey what changes are being made. While it references 'stamps' (likely timestamps), it lacks specificity about the nature of the fix. Consider revising the title to be more descriptive, such as 'Normalize date fields for Appwrite migration documents' to better reflect the actual changes.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@abnegate abnegate merged commit 5e5deb9 into main Jan 20, 2026
4 checks passed
@abnegate abnegate deleted the fix-import-stamps branch January 20, 2026 06:12
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.

2 participants