Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(site): allow only template admins/owners to navigate to template version pages by clicking VersionRow - #27550

Merged
aqandrew merged 8 commits into
mainfrom
aqandrew/devex-486-versions-page-for-template-members-shows-404-error
Aug 4, 2026
Merged

fix(site): allow only template admins/owners to navigate to template version pages by clicking VersionRow#27550
aqandrew merged 8 commits into
mainfrom
aqandrew/devex-486-versions-page-for-template-members-shows-404-error

Conversation

@aqandrew

@aqandrew aqandrew commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

fixes DEVEX-486

From TemplateVersionsPage (/templates/:organization/:template/versions), users navigate to individual template version pages (TemplateVersionPage, i.e., /templates/:organization/:template/versions/:version) by clicking VersionRow.

This PR makes this click-to-navigate behavior on VersionRow available to admin/owner users only, per @matifali's suggestion

@aqandrew
aqandrew requested review from aslilac and jakehwll July 27, 2026 21:26
@linear-code

linear-code Bot commented Jul 27, 2026

Copy link
Copy Markdown

DEVEX-486

@aqandrew aqandrew changed the title fix: allow only template admins/owners to navigate to template version pages fix(site): allow only template admins/owners to navigate to template version pages Jul 27, 2026
@aqandrew
aqandrew marked this pull request as ready for review July 28, 2026 19:19
@aqandrew
aqandrew marked this pull request as draft July 30, 2026 22:13
@aqandrew
aqandrew marked this pull request as ready for review July 30, 2026 22:41
@aqandrew

Copy link
Copy Markdown
Contributor Author

I initially got the ternary logic backwards + broke the VersionsTable stories. Just fixed those so this is good to review now!

@matifali matifali left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we should still allow navigation to this page but just make those version rows unclickable for non authorized users.

There is value in seeing this history and what current version a template is on for all users of the template

@aqandrew

aqandrew commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

I think we should still allow navigation to this page but just make those version rows unclickable for non authorized users.

@matifali For sure! Making the version rows unclickable for non-admins/owners is exactly what this PR does. I think the title/description might be misleading / not specific enough, so I'll adjust them

matifali commented Aug 4, 2026

Copy link
Copy Markdown
Member

Yes the title is misleading.

Thanks for explaining it.

@matifali
matifali dismissed their stale review August 4, 2026 19:53

resolved

@aqandrew aqandrew changed the title fix(site): allow only template admins/owners to navigate to template version pages fix(site): allow only template admins/owners to navigate to template version pages by clicking VersionRow Aug 4, 2026
@aqandrew
aqandrew merged commit ca3de3c into main Aug 4, 2026
43 checks passed
@aqandrew
aqandrew deleted the aqandrew/devex-486-versions-page-for-template-members-shows-404-error branch August 4, 2026 20:27
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 4, 2026
@matifali

matifali commented Aug 5, 2026

Copy link
Copy Markdown
Member

Verified the fix on dev.coder.com

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants