Skip to content

Added custom serializer for table variation#311

Merged
luca-bellenghi merged 4 commits intomainfrom
us_69610_listing_table
Jul 31, 2025
Merged

Added custom serializer for table variation#311
luca-bellenghi merged 4 commits intomainfrom
us_69610_listing_table

Conversation

@pnicolli
Copy link
Member

No description provided.

@pnicolli pnicolli self-assigned this Jul 31, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a custom serializer for table variations in search and listing blocks. The serializer enhances the block data by adding field properties from content type schemas to table columns.

  • Introduces two new block serializers: SearchTableVariationBlockSerialize and ListingTableVariationBlockSerialize
  • Adds schema lookup functionality to retrieve field properties for content types referenced in table columns
  • Registers the serializers in the ZCML configuration to make them active

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/design/plone/contenttypes/restapi/serializers/tablevariation.py Implements the core serializer classes with schema lookup and field property enhancement
src/design/plone/contenttypes/restapi/serializers/configure.zcml Registers the new serializers as subscribers for block field serialization
CHANGES.rst Documents the new feature in the changelog

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@coveralls
Copy link

coveralls commented Jul 31, 2025

Pull Request Test Coverage Report for Build 16651518809

Details

  • 24 of 45 (53.33%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 68.65%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/design/plone/contenttypes/restapi/serializers/tablevariation.py 24 45 53.33%
Totals Coverage Status
Change from base Build 14711416540: -0.1%
Covered Lines: 3381
Relevant Lines: 4925

💛 - Coveralls

@pnicolli
Copy link
Member Author

Il security check non so cosa sia e mi sembra fallire perché non riesce a buildarsi (usa debian buster in docker e non trova gli indici dei pacchetti di sistema)

@luca-bellenghi luca-bellenghi merged commit bf277d5 into main Jul 31, 2025
12 of 14 checks passed
@luca-bellenghi luca-bellenghi deleted the us_69610_listing_table branch July 31, 2025 14:40
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