fix(intergral/dashboards): refresh permissions before redirect after save#58
Open
jhawksley-intergral wants to merge 5 commits into12.3.x-intergralfrom
Open
fix(intergral/dashboards): refresh permissions before redirect after save#58jhawksley-intergral wants to merge 5 commits into12.3.x-intergralfrom
jhawksley-intergral wants to merge 5 commits into12.3.x-intergralfrom
Conversation
…save Await contextSrv.fetchUserPermissions() before navigating to a newly saved dashboard. Without this, the redirect can hit stale cached permissions, causing a transient "not allowed to view" error. - Add contextSrv import - Await permission refresh before locationService.replace() Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jhawksley-intergral
had a problem deploying
to
Docker Hub
GitHub Actions
Failure
Use github.head_ref for PRs (source branch name) instead of github.ref_name (which resolves to the merge ref e.g. "58/merge"). Add Compute image tag step to replace / with - in branch names. Matches the workflow from feat/alerting-partitioner. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jhawksley-intergral
temporarily deployed
to
Docker Hub
GitHub Actions
Inactive
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jhawksley-intergral
temporarily deployed
to
Docker Hub
GitHub Actions
Inactive
…board create - Expand ClearUserPermissionCache to also clear basic role caches (Editor/Viewer) so managed role permissions for newly created dashboards are picked up on the next request - Add ClearCacheFor method to Resolvers for targeted scope cache invalidation Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…irect after save In multi-instance deployments, the instance serving the redirect after saving a new dashboard may not have the dashboard or its permissions propagated yet, resulting in "Not found" or "Not allowed to view". - Add waitForDashboardReady() that polls the dashboard DTO API (up to 8 attempts, 500ms apart) before redirecting - Applies to new dashboards and save-as-copy Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jhawksley-intergral
requested a deployment
to
Docker Hub
GitHub Actions
In progress
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
contextSrv.fetchUserPermissions()beforelocationService.replace()in the save handler🤖 Generated with Claude Code