Skip to content

Re-jig the navigation to remove/move items - #312

Merged
ianmiell merged 1 commit into
mainfrom
imiell-navigation-reduction
Sep 24, 2026
Merged

ianmiell merged 1 commit into
mainfrom
imiell-navigation-reduction

Conversation

@ianmiell

Copy link
Copy Markdown
Contributor

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 6676cf0f-061d-4c41-a410-aa2fbb8c6191


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.

Comment thread src/views/LeftSideNav.vue Outdated
Comment on lines +193 to +201
// name: 'users-list',
// title: 'System Users',
// permission: ADMIN_MANAGE,
// },
// {
// name: 'admin-groups',
// title: 'Groups',
// permission: ADMIN_MANAGE,
// },

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These are probably still relevant - it's how you add permissions to users / groups

Comment thread src/views/LeftSideNav.vue Outdated
Comment on lines +202 to +206
// {
// name: 'admin-agents',
// title: 'Agents',
// permission: ADMIN_MANAGE,
// },

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also relevant - Agent SA + Service Account Keys CRUD

Comment thread src/views/LeftSideNav.vue Outdated
Comment on lines +218 to +228
// name: 'admin-diagnostics',
// title: 'Diagnostics',
// abbr: 'DIAG',
// permission: ADMIN_MANAGE,
// },
// {
// name: 'admin-import',
// title: 'Import',
// abbr: 'IMP',
// permission: ADMIN_MANAGE,
// },

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Both important - Diagnostics serves to debug issues that are hardly seen via logs; Import I just use it so much :((

Comment thread src/constants/ssp-section-tabs.ts Outdated
{ label: 'System Implementation', suffix: 'implementation' },
// { label: 'System Characteristics', suffix: 'characteristics' },
// { label: 'System Implementation', suffix: 'implementation' },
{ label: 'Users', suffix: 'users' },

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we can probably remove system users, IMO

Comment thread src/views/LeftSideNav.vue Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we should remove this as well.

Comment thread src/views/LeftSideNav.vue Outdated
@@ -161,71 +119,113 @@ const links = ref<Array<NavigationItem>>([

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

At least for now - assessment-plan, assessment-result and poem could also be removed (maybe the whole Implementation tbh

@ianmiell
ianmiell force-pushed the imiell-navigation-reduction branch 4 times, most recently from dd3439b to da153f3 Compare September 23, 2026 16:56
@ianmiell
ianmiell force-pushed the imiell-navigation-reduction branch from da153f3 to 2e0f4bc Compare September 23, 2026 17:04
@ianmiell
ianmiell merged commit d9985e0 into main Sep 24, 2026
3 checks passed
@ianmiell
ianmiell deleted the imiell-navigation-reduction branch September 24, 2026 10:01
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.

2 participants