Skip to content

OTWO-7740 Update admin dashboard link - #1956

Open
kaushak-ai wants to merge 1 commit into
mainfrom
OTWO-7740
Open

kaushak-ai wants to merge 1 commit into
mainfrom
OTWO-7740

Conversation

@kaushak-ai

@kaushak-ai kaushak-ai commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

No description provided.

@kaushak-ai
kaushak-ai requested review from Niharika1117 and Priya5 and a balanced review from Copilot October 13, 2026 11:51
@kaushak-ai kaushak-ai self-assigned this Oct 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The redirect bypasses ActiveAdmin authorization because the destination dashboard lacks an admin access guard.

1 open finding
What changed in this PR

Redirects the legacy admin dashboard route to the new dashboard and updates navigation accordingly.

Changes:

  • Redirects /admin/dashboard to /oh_admin.
  • Updates the Admin navigation link to the new dashboard.
File Description
config/​routes.rb Adds the legacy dashboard redirect.
app/​views/​layouts/​partials/​_mast.html.haml Points Admin navigation to /oh_admin.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread config/routes.rb Outdated
Comment thread config/routes.rb Outdated
@Priya5

Priya5 commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Please update the PR title to OTWO-7740 Update admin dashboard link to keep it compact and for better readability

@kaushak-ai kaushak-ai changed the title OTWO-7740: Added a redirect so visiting /admin/dashboard now redirect to /oh_admin. Also updated old admin link to new one OTWO-7740 Update admin dashboard link Oct 13, 2026
Comment thread app/views/layouts/partials/_mast.html.haml Outdated
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.

3 participants