Skip to content

Conversation

@edwardgou-sentry
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 28, 2025
@edwardgou-sentry edwardgou-sentry marked this pull request as ready for review November 28, 2025 19:11
@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner November 28, 2025 19:11
@edwardgou-sentry edwardgou-sentry requested a review from a team November 28, 2025 19:16
dashboardPermissions,
dashboardCreator
) && !isPrebuiltDashboard;

Copy link
Contributor

Choose a reason for hiding this comment

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

Small nit, now the name hasEditAccess is a bit misleading imo because it now also checks against the prebuilt dashboard variable. Perhaps something like canEdit makes more sense

@edwardgou-sentry edwardgou-sentry merged commit 25ed920 into master Dec 1, 2025
49 checks passed
@edwardgou-sentry edwardgou-sentry deleted the egou/fix/prebuilt-dashboard-no-edit-permission branch December 1, 2025 21:26
jerryzhou196 pushed a commit that referenced this pull request Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants