Skip to content

Conversation

@andrest50
Copy link
Contributor

@andrest50 andrest50 commented Dec 5, 2025

Ticket

LFXV2-887

Summary

  • Implement V1MeetingRSVPEnricher to handle V1 meeting RSVP messages
  • Use meeting_id for meeting-level access control permissions
  • Set access relation to "auditor" for read-only RSVP access
  • Set history relation to "writer" for modification tracking
  • Extract email field as sort name and alias for searchability

Changes

  • Created v1_meeting_rsvp_enricher.go with custom access control logic
  • Added ObjectTypeV1MeetingRSVP constant to messaging constants
  • Registered new enricher in IndexerService

🤖 Generated with Claude Code

Implement V1MeetingRSVPEnricher to handle V1 meeting RSVP messages:
- Uses meeting_id for access control object (meeting-level permissions)
- Sets access check relation to "auditor" for read-only RSVP access
- Sets history check relation to "writer" for modification tracking
- Extracts email field as sort name and alias for searchability
- Registers new enricher in IndexerService

Add ObjectTypeV1MeetingRSVP constant to messaging constants.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Signed-off-by: Andres Tobon <[email protected]>
@coderabbitai
Copy link

coderabbitai bot commented Dec 5, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch andrest50/v1-meeting-rsvp

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

Signed-off-by: Andres Tobon <[email protected]>
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