-
Notifications
You must be signed in to change notification settings - Fork 111
Docs for the push activity tracking #984
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
base: main
Are you sure you want to change the base?
Conversation
|
Warning Rate limit exceeded@Aviatorscode2 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 3 minutes and 3 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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. 📒 Files selected for processing (1)
WalkthroughAdds a new documentation page "push-activity-tracking" describing a client→backend→Novu workflow for forwarding push interaction events, and updates the push integration metadata to include the new page. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant U as Client App
participant B as App Backend
participant N as Novu API
participant F as Novu Activity Feed
rect rgba(200,230,255,0.25)
note over U: Step 2 — capture push interaction on device
U->>B: POST /track-push {eventType,eventId,timestamp,content,deviceId,platform}
end
rect rgba(220,255,220,0.25)
note over B: Step 3 — backend forwards to Novu
B->>N: trackActivity({environmentId,integrationId,requestBody})
N-->>B: HTTP 2xx/4xx
end
rect rgba(255,240,200,0.25)
note over N,F: Step 4 — event appears in Activity Feed
N-->>F: store/display event
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (2 passed)
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. Comment |
👷 Deploy Preview for docs-novu processing.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
public/images/channels-and-providers/push/activity-tracking/enable-activity-tracking.pngis excluded by!**/*.png
📒 Files selected for processing (2)
content/docs/platform/integrations/push/meta.json(1 hunks)content/docs/platform/integrations/push/push-activity-tracking.mdx(1 hunks)
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
….mdx Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
….mdx Co-authored-by: DianaHackmamba <diana@hackmamba.io>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
🧹 Nitpick comments (2)
content/docs/platform/integrations/push/push-activity-tracking.mdx (2)
7-9: Fix the incomplete sentence in the introduction.Line 9 is a sentence fragment that disrupts the flow. It should be restructured to form a complete sentence or better connected to the previous thought.
Suggested fix:
- Where your application captures and forwards push notification events to the Novu. Once received, Novu processes and displays these events alongside other channels for a unified tracking experience. + This is where your application captures and forwards push notification events to Novu. Once received, Novu processes and displays these events alongside other channels for a unified tracking experience.
151-151: Break up the long concluding sentence for readability.The final sentence is dense and could be split into two shorter sentences for better flow:
- Once these steps are completed, your application will send push notification engagement data to Novu, giving you a complete, unified view of your notification performance in the Activity Feed. + Once these steps are completed, your application will send push notification engagement data to Novu. This gives you a complete, unified view of your notification performance in the Activity Feed.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/docs/platform/integrations/push/push-activity-tracking.mdx(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Build and Lint
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
content/docs/platform/integrations/push/push-activity-tracking.mdx
Outdated
Show resolved
Hide resolved
….mdx Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Summary by CodeRabbit