Skip to content

Conversation

@damienmiheev
Copy link

@damienmiheev damienmiheev commented Jan 14, 2026

This change adds the required context field to TriggerEventRequest to stay compatible with the latest Novu API updates.

Summary by CodeRabbit

  • New Features
    • Event trigger functionality now supports passing additional context information.

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 14, 2026

📝 Walkthrough

Walkthrough

A new context field of type Map<String, Object> was added to the TriggerEventRequest class to enable additional contextual information to be passed when triggering events.

Changes

Cohort / File(s) Summary
Event Request Enhancement
src/main/java/co/novu/api/events/requests/TriggerEventRequest.java
Added context field (Map<String, Object>) to support additional contextual data in event triggering

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny field hops into place,
Context now joins the event race,
Maps and Objects, side by side,
More data flows with springy stride!

🚥 Pre-merge checks | ✅ 3
✅ 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 'Add context property to TriggerEventRequest' directly and clearly describes the main change: adding a context field to the TriggerEventRequest class, which aligns with the file modifications and PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9003047 and 6bc7dc2.

📒 Files selected for processing (1)
  • src/main/java/co/novu/api/events/requests/TriggerEventRequest.java
🔇 Additional comments (1)
src/main/java/co/novu/api/events/requests/TriggerEventRequest.java (1)

29-29: LGTM! The new context field follows the existing patterns and matches the Novu API specification.

The field type Map<String, Object> correctly represents the context object (with context type names as keys and context references/data as values). Positioning and implementation are consistent with similar fields like payload and overrides. Lombok's @Data annotation will generate the necessary getters/setters.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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.

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