Skip to content

Fix: Add notification channel navigation entries for IS 7.0.0 and 7.1.0 (Product IS issue #26941)#5941

Open
wso2-engineering-bot wants to merge 1 commit intomasterfrom
fixing-product-is-issue-26941-1772903150
Open

Fix: Add notification channel navigation entries for IS 7.0.0 and 7.1.0 (Product IS issue #26941)#5941
wso2-engineering-bot wants to merge 1 commit intomasterfrom
fixing-product-is-issue-26941-1772903150

Conversation

@wso2-engineering-bot
Copy link

@wso2-engineering-bot wso2-engineering-bot commented Mar 7, 2026

This PR was automatically generated by Claude AI.

Changes Made

IS 7.1.0

Added a Notification Channels section to the navigation with links to:

  • Configure SMS Provider
  • Configure Email Provider
  • Configure Push Provider

IS 7.0.0

Added an Email & SMS section to the navigation with links to:

  • Configure SMS Provider
  • Configure Email Provider

(Note: IS 7.0.0 uses "Email & SMS" menu item in the console, not "Notification Channels")

Files Changed

  • en/identity-server/7.1.0/mkdocs.yml - Added Notification Channels navigation section
  • en/identity-server/7.0.0/mkdocs.yml - Added Email & SMS navigation section

All referenced documentation files already exist and are accessible. The changes ensure that the navigation structure matches IS 7.2.0 which already has the correct navigation.

Summary by CodeRabbit

Documentation

  • Added new guides for configuring SMS, Email, and Push notification providers across identity server versions 7.0.0 and 7.1.0
  • Expanded navigation to include dedicated notification channels section for improved discoverability

….0 (product-is#26941)

- Added 'Notification Channels' section to IS 7.1.0 navigation with SMS, Email, and Push provider configuration links
- Added 'Email & SMS' section to IS 7.0.0 navigation with SMS and Email provider configuration links
- Resolves missing navigation entries reported in product-is issue #26941

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


wso2-engineering-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 7, 2026

Walkthrough

Navigation entries for SMS, Email, and Push notification channel configuration guides are added to mkdocs.yml configuration files in Identity Server 7.0.0 and 7.1.0 documentation. These guides now appear in the left-side navigation bar, making previously undiscoverable content accessible to users.

Changes

Cohort / File(s) Summary
Documentation Navigation Updates
en/identity-server/7.0.0/mkdocs.yml, en/identity-server/7.1.0/mkdocs.yml
Added notification channel configuration guides to navigation structure. Version 7.0.0 adds "Email & SMS" subsection under Guides; version 7.1.0 adds "Notification Channels" section with SMS, Email, and Push provider configuration links.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hopping through the navigation trees,
Guides now visible with such ease!
SMS, Email, Push—no longer lost,
Sidebar shine at minimal cost! 📍✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description provides clear context about the fix, explains the differences between versions, lists changed files, and verifies completion. However, it does not follow the repository's template structure with Purpose, Related PRs, Test environment, and Security checks sections. Consider restructuring the description to match the repository's template sections, particularly adding Test environment details and security check confirmations.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding missing notification channel navigation entries for IS 7.0.0 and 7.1.0, and references the specific issue being fixed.
Linked Issues check ✅ Passed The PR fully addresses issue #26941 by adding the missing notification channel navigation entries for both IS 7.1.0 and 7.0.0, matching the structure in IS 7.2.0 and accounting for the IS 7.0.0 console naming convention.
Out of Scope Changes check ✅ Passed All changes are directly scoped to the linked issue: adding navigation entries for notification channels in mkdocs.yml files for the two specified versions. No unrelated modifications are present.
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fixing-product-is-issue-26941-1772903150

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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

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 `@en/identity-server/7.0.0/mkdocs.yml`:
- Around line 972-974: The "Email & SMS" nav block is currently indented under
"Connectors -> Onfido" causing the links to appear in the wrong sidebar
location; move the whole block titled "Email & SMS" (and its children "Configure
SMS Provider: guides/notification-channels/configure-sms-provider.md" and
"Configure Email Provider: deploy/configure/email-sending-module.md") out one
indentation level so it is not a child of the "Onfido" connector, and then add
the same "Email & SMS" block at the correct level under the top-level "Guides"
section (near the other guide categories) so the links appear in the intended
Guides navigation.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4465d190-d5f5-45c1-be83-263fed708091

📥 Commits

Reviewing files that changed from the base of the PR and between 3f332c6 and 1a3589c.

📒 Files selected for processing (2)
  • en/identity-server/7.0.0/mkdocs.yml
  • en/identity-server/7.1.0/mkdocs.yml

Comment on lines +972 to +974
- Email & SMS:
- Configure SMS Provider: guides/notification-channels/configure-sms-provider.md
- Configure Email Provider: deploy/configure/email-sending-module.md
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Move this section out of Connectors > Onfido.

At the current indentation, Email & SMS becomes a child of Connectors -> Onfido, not a Guides entry. That means the new links land in the wrong part of the sidebar and the 7.0.0 navigation gap remains effectively unfixed.

Suggested fix
-      - Email & SMS:
-          - Configure SMS Provider: guides/notification-channels/configure-sms-provider.md
-          - Configure Email Provider: deploy/configure/email-sending-module.md

Then add the same block under the Guides section at the intended level, e.g. near the other guide categories:

+      - Email & SMS:
+          - Configure SMS Provider: guides/notification-channels/configure-sms-provider.md
+          - Configure Email Provider: deploy/configure/email-sending-module.md
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@en/identity-server/7.0.0/mkdocs.yml` around lines 972 - 974, The "Email &
SMS" nav block is currently indented under "Connectors -> Onfido" causing the
links to appear in the wrong sidebar location; move the whole block titled
"Email & SMS" (and its children "Configure SMS Provider:
guides/notification-channels/configure-sms-provider.md" and "Configure Email
Provider: deploy/configure/email-sending-module.md") out one indentation level
so it is not a child of the "Onfido" connector, and then add the same "Email &
SMS" block at the correct level under the top-level "Guides" section (near the
other guide categories) so the links appear in the intended Guides navigation.

@AnuradhaSK
Copy link
Contributor

@copilot In 7.0.0 mkdocs.yaml, No need to mention Email & SMS as navigation bar title. Use notification channel.
My point is Configurations related to SMS providers are located under the Notification Channels section. such content should be changed to under Email & SMS instead of under the Notification Channels section in the content.

Further, For both IS-7.0.0 and IS-7.1.0 email publisher doc should also come under guides/notification-channels/configure-email-provider/ path. If the file is not in the relevant folder need to get from includes

Copy link

Copilot AI commented Mar 8, 2026

@AnuradhaSK I've opened a new pull request, #5943, to work on those changes. Once the pull request is ready, I'll request review from you.

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.

SMS, Email, Push notification channel configuration guide is not in documentation side navigation bar

4 participants