Fix capitalization issue in the inbox section#1051
Merged
jainpawan21 merged 4 commits intomainfrom Feb 28, 2026
Merged
Conversation
Contributor
📝 WalkthroughWalkthroughThis PR renames the Inbox section identifier from "advanced-concepts" to "advanced-features", updates several doc titles/frontmatter for capitalization consistency, and adds two redirect mappings for the renamed advanced-features paths. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
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 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Member
|
Look into the merge conflict |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@content/docs/platform/inbox/configuration/inbox-with-context.mdx`:
- Around line 2-3: The frontmatter key pageTitle is using sentence case ("Use
contexts for Inbox personalization") while title is Title Case; change the
pageTitle value to Title Case to match title (e.g., "Use Contexts for Inbox
Personalization") so both pageTitle and title are consistent; update the
pageTitle string in the frontmatter to the Title Case version.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
content/docs/platform/inbox/advanced-features/localization.mdxcontent/docs/platform/inbox/advanced-features/meta.jsoncontent/docs/platform/inbox/advanced-features/multi-tenancy.mdxcontent/docs/platform/inbox/configuration/inbox-with-context.mdxcontent/docs/platform/inbox/configuration/meta.jsoncontent/docs/platform/inbox/configuration/styling.mdxcontent/docs/platform/inbox/meta.json
content/docs/platform/inbox/configuration/inbox-with-context.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Member
|
Please update middleware redirects for path and file name changes |
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/middleware.ts`:
- Around line 81-85: The redirect for '/platform/inbox/react/localization'
currently points to the intermediate
'/platform/inbox/advanced-concepts/localization', creating a chain because
'/platform/inbox/advanced-concepts/localization' itself redirects to
'/platform/inbox/advanced-features/localization'; update the mapping so
'/platform/inbox/react/localization' maps directly to
'/platform/inbox/advanced-features/localization' to eliminate the double
redirect (adjust the entry where the key '/platform/inbox/react/localization' is
defined).
Comment on lines
81
to
+85
| '/platform/inbox/react/localization': '/platform/inbox/advanced-concepts/localization', | ||
| '/platform/inbox/advanced-concepts/localization': | ||
| '/platform/inbox/advanced-features/localization', | ||
| '/platform/inbox/advanced-concepts/multi-tenancy': | ||
| '/platform/inbox/advanced-features/multi-tenancy', |
Contributor
There was a problem hiding this comment.
Redirect chain detected: update the existing redirect at Line 81.
Line 81 redirects /platform/inbox/react/localization → /platform/inbox/advanced-concepts/localization, but the new redirect at Lines 82-83 then redirects that path to /platform/inbox/advanced-features/localization. This creates a double redirect.
Update Line 81 to point directly to the final destination to avoid the redirect chain.
🔧 Proposed fix to eliminate the redirect chain
- '/platform/inbox/react/localization': '/platform/inbox/advanced-concepts/localization',
+ '/platform/inbox/react/localization': '/platform/inbox/advanced-features/localization',🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/middleware.ts` around lines 81 - 85, The redirect for
'/platform/inbox/react/localization' currently points to the intermediate
'/platform/inbox/advanced-concepts/localization', creating a chain because
'/platform/inbox/advanced-concepts/localization' itself redirects to
'/platform/inbox/advanced-features/localization'; update the mapping so
'/platform/inbox/react/localization' maps directly to
'/platform/inbox/advanced-features/localization' to eliminate the double
redirect (adjust the entry where the key '/platform/inbox/react/localization' is
defined).
jainpawan21
approved these changes
Feb 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@jainpawan21, kindly merge this. I fix capitalization issues in the Inbox section to match how other headings are arranged (Using title case on page titles)
Summary by CodeRabbit
Documentation
Chores