Skip to content

Conversation

@Saurabhkmr98
Copy link
Member

@Saurabhkmr98 Saurabhkmr98 commented Dec 15, 2025

Description

  • Added description_html description description_binary fields in Pages External API Serializer

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Improvement (change that would cause existing functionality to not work as expected)
  • Code refactoring
  • Performance improvements
  • Documentation update

Screenshots and Media (if applicable)

Test Scenarios

References

Summary by CodeRabbit

Release Notes

  • New Features
    • Pages now support extended content descriptions, including HTML formatting, binary content, and flexible description fields for richer content management.

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

@makeplane
Copy link

makeplane bot commented Dec 15, 2025

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@coderabbitai
Copy link

coderabbitai bot commented Dec 15, 2025

Warning

Rate limit exceeded

@Saurabhkmr98 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 16 minutes and 45 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between a27513b and 9544b06.

📒 Files selected for processing (1)
  • pyproject.toml (1 hunks)

Walkthrough

Three new optional fields were added to the Page model in plane/models/pages.py: description_html, description_binary, and description. This expands the model's schema to provide multiple representations of page descriptions without modifying existing fields or behavior.

Changes

Cohort / File(s) Change Summary
Page Model Schema Expansion
plane/models/pages.py
Added three new optional fields to the Page class: description_html (str | None), description_binary (str | None), and description (dict | str | None) for enhanced description handling

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Verify that the new fields are properly handled in model serialization/deserialization
  • Check for any required database migrations or schema updates
  • Confirm type annotations are consistent with existing patterns in the codebase

Poem

🐰 Three new fields, so neat and bright,
Description's now a fuller sight,
HTML, binary, rich or plain—
Our Page model's got more brain!
Hop, hop, hooray for schema's gain! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding missing description-related fields (description_html, description_binary, description) to the Pages model.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

@Prashant-Surya Prashant-Surya merged commit 2a1feba into main Dec 24, 2025
1 check passed
@Prashant-Surya Prashant-Surya deleted the fix-missing_field_page_api branch December 24, 2025 12:22
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.

3 participants