Skip to content

Fix capitalization issue in the inbox section #1051

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.

Sign up for GitHub

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

Merged
jainpawan21 merged 4 commits into from
Feb 28, 2026
Merged

Fix capitalization issue in the inbox section #1051

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
38d1a88
Fix capitalization issue in the inbox section
Aviatorscode2 Feb 16, 2026
25ff4ce
Merge branch 'main' of https://github.com/novuhq/docs into fix-capita…
Aviatorscode2 Feb 27, 2026
93007c2
Update content/docs/platform/inbox/configuration/inbox-with-context.mdx
Aviatorscode2 Feb 27, 2026
bb015e9
Update middleware redirects for localization and multi-tenancy
Aviatorscode2 Feb 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
0 .../inbox/advanced-concepts/localization.mdx → .../inbox/advanced-features/localization.mdx
View file
Open in desktop
File renamed without changes.
2 changes: 1 addition & 1 deletion ...latform/inbox/advanced-concepts/meta.json → ...latform/inbox/advanced-features/meta.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"title": "Advanced Concepts",
"title": "Advanced Features",
"icon": "Network",
"pages": ["localization", "multi-tenancy"],
"description": "Learn how to implement localization and tenant management in the Inbox component."
Expand Down
0 ...inbox/advanced-concepts/multi-tenancy.mdx → ...inbox/advanced-features/multi-tenancy.mdx
View file
Open in desktop
File renamed without changes.
4 changes: 2 additions & 2 deletions content/docs/platform/inbox/configuration/inbox-with-context.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
pageTitle: 'Using contexts for Inbox personalization'
title: 'Inbox with context'
pageTitle: 'Use Contexts for Inbox Personalization'
title: 'Inbox with Context'
description: 'Learn how to use contexts in the Inbox component to filter and personalize notifications for your subscribers.'
icon: 'Package'
---
Expand Down
2 changes: 1 addition & 1 deletion content/docs/platform/inbox/configuration/meta.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"title": "Customize and configure",
"title": "Customize and Configure",
"icon": "SlidersHorizontal",
"pages": ["styling", "tabs", "preferences", "data-object", "inbox-with-context", "icons"],
"description": "Learn how to configure your inbox with styling, tabs, preferences, data objects, icons and context functionality"
Expand Down
2 changes: 1 addition & 1 deletion content/docs/platform/inbox/configuration/styling.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
pageTitle: 'Styling the Inbox component'
title: 'Styling and theming'
title: 'Styling and Theming'
description: 'Learn how to style the pre built Inbox component'
icon: 'Palette'
---
Expand Down
2 changes: 1 addition & 1 deletion content/docs/platform/inbox/meta.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"features",
"configuration",
"advanced-customization",
"advanced-concepts",
"advanced-features",
"headless-mode",
"prepare-for-production",
"migration-guide"
Expand Down
4 changes: 4 additions & 0 deletions src/middleware.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,10 @@ export default async function middleware(request: NextRequest, event: NextFetchE
'/platform/inbox/react/styling#appearance-prop': '/platform/inbox/configuration/styling',
'/platform/inbox/react/headless': '/platform/inbox/headless-mode',
'/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',
Comment on lines 81 to +85
Copy link
Contributor

@coderabbitai coderabbitai bot Feb 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

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).


// Workflow section (old → new structure)
'/platform/workflow/layouts':
Expand Down