fix(databases): remove leftover per-table Usage tab#3130
Closed
HarshMN2345 wants to merge 3 commits into
Closed
Conversation
Same cleanup as #3128, one level down: the per-table Usage tab still called tablesDB.getTableUsage(), which is going away with the backend usage-endpoint removal and is already showing empty data on cloud.
Contributor
Greptile SummaryThis PR removes the per-table database Usage page. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (3): Last reviewed commit: "fix(databases): remove orphaned shared e..." | Re-trigger Greptile |
| @@ -32,12 +32,6 @@ | |||
| title: 'Indexes', | |||
Contributor
There was a problem hiding this comment.
The table Usage tab and page are removed here, but the table layout still registers a command-palette action that navigates to withPath(basePath, '/usage'). A user can still trigger that action from any table page and land on the now-deleted route, showing a client-side 404.
Context Used: AGENTS.md (source)
Prompt To Fix With AI
This is a comment left during a code review.
Path: src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/header.svelte
Line: 32
Comment:
**Stale Usage Command Route**
The table Usage tab and page are removed here, but the table layout still registers a command-palette action that navigates to `withPath(basePath, '/usage')`. A user can still trigger that action from any table page and land on the now-deleted route, showing a client-side 404.
**Context Used:** AGENTS.md ([source](https://app.greptile.com/appwrite/github/appwrite/console/-/custom-context?memory=f4227d43-a98a-4882-876a-cad1bc006878))
How can I resolve this? If you propose a fix, please make it concise.The "Go to usage" (g u) shortcut pointed at the table usage route removed earlier in this branch, leaving a broken command.
Only the table usage page consumed this component; it's now dead code after removing that route.
This was referenced Jul 21, 2026
Member
Author
|
Superseded by #3134 — consolidating everything into a single PR. |
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
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
table-[table]/usage), which callstablesDB.getTableUsage()— an endpoint being dropped along with the backend usage-endpoint removal, and already showing no useful data on cloud.Test plan
bun run checkpasses with no new errors