Skip to content
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

feat: add review modal core structure #2695

Open
wants to merge 4 commits into
base: feat/reviews-and-ratings
Choose a base branch
from

Conversation

artursantiago
Copy link
Collaborator

@artursantiago artursantiago commented Feb 19, 2025

⚠️ THIS PR DEPENDS ON: ⚠️

What's the purpose of this pull request?

This PR integrates the AddReviewModal component in the core package for the Review and Ratings section.
The actual review form implementation will be handled in a separate PR to be easy to review.

How it works?

  • Adding schema configuration in sections.json for AddReviewModal properties in the Headless CMS
  • Creating a new AddReviewModal component in packages/core/src/components/reviews that will handle form logic
  • Integrating the modal within the ReviewsAndRatings UI with a temporary trigger button

How to test it?

  • Check AddReviewModal structure in core package and its types declarations
  • Confirm that CMS-configured content is properly displayed

Starters Deploy Preview

Preview

References

FEATURE BRANCH
Figma
SFS-2082

@artursantiago artursantiago mentioned this pull request Feb 19, 2025
2 tasks
@artursantiago artursantiago changed the title Feat/add review modal core structure feat: add review modal core structure Feb 19, 2025
Copy link

codesandbox-ci bot commented Feb 19, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@Guilera Guilera force-pushed the feat/reviews-and-ratings branch from cc15cdd to 33db057 Compare February 19, 2025 17:34
@artursantiago artursantiago force-pushed the feat/add-review-modal-core-structure branch from 0bd85ce to ffa6c4b Compare February 20, 2025 16:10
@artursantiago artursantiago force-pushed the feat/add-review-modal-core-structure branch from ffa6c4b to 6906fa9 Compare February 21, 2025 12:18
@artursantiago artursantiago self-assigned this Feb 21, 2025
@artursantiago artursantiago marked this pull request as ready for review February 21, 2025 16:49
@artursantiago artursantiago requested a review from a team as a code owner February 21, 2025 16:49
@artursantiago artursantiago requested review from renatamottam, lariciamota and hellofanny and removed request for a team, renatamottam and lariciamota February 21, 2025 16:49
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