fix: show project settings in read-only mode for developers#3011
fix: show project settings in read-only mode for developers#3011HarshMN2345 merged 2 commits intomainfrom
Conversation
Console (appwrite/console)Project ID: Tip Function builds can take up to 45 minutes before timing out |
Greptile SummaryThis PR exposes the project settings page to developers (users without Confidence Score: 5/5Safe to merge — changes are purely additive UI guards with no risk of data mutation for read-only users. All interactive elements across every affected component are correctly disabled for non-writers. The load function properly re-throws non-permission errors. The previous concern about Promise.allSettled silently swallowing all failures has been fully addressed with targeted 401/403 handling. No P1 or P0 issues found. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "fix: only silence readonly settings perm..." | Re-trigger Greptile |
| const variables = | ||
| variablesResult.status === 'fulfilled' | ||
| ? variablesResult.value | ||
| : { | ||
| total: 0, | ||
| variables: [] | ||
| }; | ||
|
|
||
| const installations = | ||
| installationsResult.status === 'fulfilled' | ||
| ? installationsResult.value | ||
| : { | ||
| total: 0, | ||
| installations: [] | ||
| }; |
There was a problem hiding this comment.
Lets add comment explaining why we silence errors - what kind of error is intentionally silenced. If possible, let's scope it to that error only

What does this PR do?
(Provide a description of what this PR does.)
Test Plan
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?
(Write your answer here.)