Skip to content

fix(databases): remove leftover per-table Usage tab#3130

Closed
HarshMN2345 wants to merge 3 commits into
mainfrom
fix-remove-table-usage-tab
Closed

fix(databases): remove leftover per-table Usage tab#3130
HarshMN2345 wants to merge 3 commits into
mainfrom
fix-remove-table-usage-tab

Conversation

@HarshMN2345

Copy link
Copy Markdown
Member

Summary

Test plan

  • bun run check passes with no new errors
  • Table page no longer shows a Usage tab

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.
@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes the per-table database Usage page. The main changes are:

  • Deleted the table Usage route and load function.
  • Removed the table header Usage tab.
  • Removed the table command shortcut for Usage.
  • Removed the unused shared Usage view and export.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/+layout.svelte Removes the table command shortcut that pointed to the deleted Usage route.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/header.svelte Removes the table header Usage tab.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/usage/[[period]]/+page.svelte Deletes the per-table Usage page component.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/usage/[[period]]/+page.ts Deletes the per-table Usage data load that called getTableUsage().
src/routes/(console)/project-[region]-[project]/databases/database-[database]/(entity)/index.ts Removes the unused Usage export.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/(entity)/views/usage/view.svelte Deletes the shared Usage view used by the removed table route.

Reviews (3): Last reviewed commit: "fix(databases): remove orphaned shared e..." | Re-trigger Greptile

@@ -32,12 +32,6 @@
title: 'Indexes',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 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)

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.

Fix in Claude Code Fix in Codex

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.
@HarshMN2345

Copy link
Copy Markdown
Member Author

Superseded by #3134 — consolidating everything into a single PR.

@HarshMN2345
HarshMN2345 deleted the fix-remove-table-usage-tab branch July 21, 2026 12:32
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