Repository navigation
docs(marketplace): point the Repositories view to Admin - #700
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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 cozystack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
## What this PR does Connecting an External-Apps repository creates a cluster-scoped Tap and runs its charts in the management cluster. That is an operator action, so the Repositories view moves from the Marketplace sidebar to Admin, under Administration. The page is at `/admin/taps` now. `/marketplace/taps` redirects there, otherwise the `:appName` catch-all would render an order page for an app called "taps". Who can open it doesn't change. The entry was ungated in the Marketplace and stays ungated in Administration, Tap RBAC decides the rest. ### Screenshots <!-- REQUIRED for UI changes: attach screenshots or screen recordings demonstrating the visual impact of your changes. PRs with UI changes without screenshots will not be merged. --> ### Downstream repositories - [ ] No downstream repository is affected by this change - [x] [cozystack/website](https://github.com/cozystack/website) - follow-up: cozystack/website#700 - [ ] [cozystack/terraform-provider-cozystack](https://github.com/cozystack/terraform-provider-cozystack) - follow-up: - [ ] [cozystack/ansible-cozystack](https://github.com/cozystack/ansible-cozystack) - follow-up: - [ ] [cozystack/ccp](https://github.com/cozystack/ccp) - follow-up: - [ ] [cozystack/talm](https://github.com/cozystack/talm) - follow-up: - [ ] [cozystack/cozyhr](https://github.com/cozystack/cozyhr) - follow-up: - [ ] [cozystack/cozy-proxy](https://github.com/cozystack/cozy-proxy) - follow-up: - [ ] [cozystack/cozystack-telemetry-server](https://github.com/cozystack/cozystack-telemetry-server) - follow-up: - [ ] [cozystack/external-apps-example](https://github.com/cozystack/external-apps-example) - follow-up: - [ ] [cozystack/examples](https://github.com/cozystack/examples) - follow-up: - [ ] [cozystack/community](https://github.com/cozystack/community) - follow-up: `content/en/docs/next/marketplace/connecting.md` on the website told readers to open the Repositories view "from the sidebar", inside the Marketplace docs. The follow-up points them to Admin and waits for this PR. ### Release note ```release-note feat(dashboard): the Repositories view moved from the Marketplace sidebar to Admin, under Administration. The old /marketplace/taps URL redirects to /admin/taps. ``` <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Added Repositories to the Administration area at `/admin/taps`. - Added Repositories to the Administration sidebar with a dedicated navigation entry. - Repositories remain accessible without requiring access to the operator area. - **Bug Fixes** - Legacy Marketplace Repositories links now redirect to `/admin/taps`. - Removed Repositories from the Marketplace sidebar to reflect its new location. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Aleksei Sviridkin (lexfrei)
left a comment
Aleksei Sviridkin (lexfrei)
left a comment
There was a problem hiding this comment.
Andrei Kvapil (@kvaps) NOT LGTM. The new path in the guide is right, but one sentence in the commit message has to go.
I checked the path against cozystack main at 52f542c4. The Admin tab is shown to everyone (useCanSeeAdmin returns true), its Administration section lists Repositories at /admin/taps (sidebar-sections.tsx:204), the button there is labelled Connect, and the old /marketplace/taps route redirects to it. The view only ships in the v1.7.0 alpha tags, so editing next/ alone is correct. No other docs page still says "from the sidebar".
The blocker is a sequencing note that would land in main's history. The body of 70ece33 ends with "Merge after the dashboard change lands." It tells reviewers when to merge, it does not explain the change, and it stays in git log long after the condition is met. The PR description has the same line ("Hold until #4286 is merged."), and this repo uses the description as the merge commit message. #4286 merged on 2026-09-17, so please drop the sentence from the commit and the line from the description.
The dashboard moves the Repositories view from the Marketplace sidebar to Admin, under Administration (cozystack/cozystack#4286). The connect guide only said to open it "from the sidebar", and it sits in the Marketplace section, so readers would look for it in the wrong place. Assisted-by: LLM Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
70ece33 to
ba8c74b
Compare
Aleksei Sviridkin (lexfrei)
left a comment
Aleksei Sviridkin (lexfrei)
left a comment
There was a problem hiding this comment.
Andrei Kvapil (@kvaps) LGTM. #4286 is merged, so I dropped the hold notes from the description and the commit message, rebased onto main, and left the change itself as it was.
2 participants
Follow-up for cozystack/cozystack#4286, which moved the dashboard Repositories view from the Marketplace sidebar to Admin, under Administration.
The connect guide only said to open the view "from the sidebar", and the page sits in the Marketplace docs, so after that change readers would look in the wrong place. Only
next/is touched, the view is not in a release yet.