Skip to content

feat: disable actions by role and show tooltip instead of hi…#2228

Open
HarshMN2345 wants to merge 2 commits into
mainfrom
feat-SER-186--Add-tooltip-where-button-is-disabled-for-better-dx
Open

feat: disable actions by role and show tooltip instead of hi…#2228
HarshMN2345 wants to merge 2 commits into
mainfrom
feat-SER-186--Add-tooltip-where-button-is-disabled-for-better-dx

Conversation

@HarshMN2345
Copy link
Copy Markdown
Member

@HarshMN2345 HarshMN2345 commented Aug 18, 2025

What does this PR do?

Replaced {#if $canWrite} gating with visible, disabled buttons + tooltip
Tooltip copy: “Your role does not allow this action”
Applied across Databases, Storage, Sites, Webhooks, Messaging, Functions, Platforms/Keys, Organization, and some Settings actions
image
image

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

yes

@appwrite
Copy link
Copy Markdown

appwrite Bot commented Aug 18, 2025

Console

Project ID: 688b7bf400350cbd60e9

Sites (2)
Site Status Logs Preview QR
 console-qa
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code
 console-cloud
688b7c18002b9b871a8f
Ready Ready View Logs Preview URL QR Code

Note

Cursor pagination performs better than offset pagination when loading further pages.

@coolify-appwrite-org
Copy link
Copy Markdown

coolify-appwrite-org Bot commented Aug 18, 2025

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2025-08-18 13:56:50 CET

@coolify-appwrite-org
Copy link
Copy Markdown

coolify-appwrite-org Bot commented Aug 18, 2025

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2025-08-18 13:56:52 CET

@HarshMN2345 HarshMN2345 requested a review from ItzNotABug August 18, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant