Skip to content

fix: unauthorized get templates fetch - #6527

Open
Schwehn42 wants to merge 2 commits into
mainfrom
js/fix/unauthorized-get-templates
Open

fix: unauthorized get templates fetch#6527
Schwehn42 wants to merge 2 commits into
mainfrom
js/fix/unauthorized-get-templates

Conversation

@Schwehn42

Copy link
Copy Markdown
Member

Description

currently, the frontend will attempt to fetch templates even if the user isn't allowed to (e.g. https://docs.scrumlr.io/self-hosting/env-vars/#allow-anonymous-template-creation).
the backend would respond with a 403.

this PR checks if the user is supposed to see custom templates and only makes the call if yes.

Changelog

  • rm duplicate template fetch in Boards
  • make template fetch conditional

Checklist

  • I have performed a self-review of my own code
  • I have written and understand every part of this contribution myself - if AI tools were used, I have thoroughly reviewed and verified all changes
  • I have commented my code, particularly in hard-to-understand areas

@Schwehn42 Schwehn42 added Review Needed This pull request is ready for review Bug (LOW) Frontend At least some of this PR changes stuff in the frontend code labels Aug 19, 2026
@github-actions

Copy link
Copy Markdown

The deployment to the dev cluster was successful. You can find the deployment here: https://6527.development.scrumlr.fra.ics.inovex.io
This deployment is only for testing purposes and will be deleted after 1 week.
To redeploy rerun the workflow.
DO NOT STORE IMPORTANT DATA ON THIS DEPLOYMENT

Deployed Images
  • ghcr.io/inovex/scrumlr.io/scrumlr-frontend:sha-a4e5ece

  • ghcr.io/inovex/scrumlr.io/scrumlr-server:sha-a4e5ece

@Schwehn42
Schwehn42 marked this pull request as ready for review August 19, 2026 09:22
@Schwehn42 Schwehn42 self-assigned this Aug 19, 2026
@Schwehn42 Schwehn42 moved this from Todo to Review in scrumlr.io Aug 19, 2026

@Planlos5000 Planlos5000 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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

Labels

Bug (LOW) Frontend At least some of this PR changes stuff in the frontend code Review Needed This pull request is ready for review

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

2 participants