Skip to content

refactor(frontend): remove Vue bridges from project database detail#19924

Merged
RainbowDashy merged 21 commits into
mainfrom
p0ny/fix-database-detail-page
Apr 9, 2026
Merged

refactor(frontend): remove Vue bridges from project database detail#19924
RainbowDashy merged 21 commits into
mainfrom
p0ny/fix-database-detail-page

Conversation

@RainbowDashy

Copy link
Copy Markdown
Contributor

Summary

  • migrate the project database detail page panels off page-level Vue bridges and onto React-native overview, changelog, revision, catalog, and schema browser components
  • replace the remaining bridge-backed catalog and schema interactions with React dialogs/tables, including grant access, table detail, revision import, and schema browsing flows
  • expand the React page test coverage and add locale entries needed by the new database detail UI

Test Plan

  • CI=1 pnpm --dir frontend test --run src/react/pages/project/ProjectDatabaseDetailPage.test.tsx src/react/pages/project/database-detail/panels/DatabaseCatalogPanel.test.tsx
  • pnpm --dir frontend fix
  • pnpm --dir frontend check
  • pnpm --dir frontend type-check

Breaking Changes

  • the project database detail experience is now fully React-owned at the page level, and the previous Vue-backed schema diagram action has been replaced by a simplified React schema browser dialog

@RainbowDashy RainbowDashy requested a review from a team as a code owner April 9, 2026 06:19
@cla-bot cla-bot Bot added the cla-signed label Apr 9, 2026
@d-bytebase d-bytebase requested a review from ecmadao April 9, 2026 06:21

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 06e77442c3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread frontend/src/react/pages/project/database-detail/panels/DatabaseRevisionPanel.tsx Outdated
@RainbowDashy RainbowDashy force-pushed the p0ny/fix-database-detail-page branch from 06e7744 to c247f5b Compare April 9, 2026 06:48

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c247f5be52

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread frontend/src/react/pages/project/database-detail/catalog/GrantAccessDialog.tsx Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ede1f11ff1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: faed0a2cb3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread frontend/src/react/pages/project/database-detail/catalog/SensitiveColumnTable.tsx Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ac4425af54

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@sonarqubecloud

sonarqubecloud Bot commented Apr 9, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
37 New issues
8 Security Hotspots
3.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@RainbowDashy RainbowDashy merged commit 79dd009 into main Apr 9, 2026
18 of 19 checks passed
@RainbowDashy RainbowDashy deleted the p0ny/fix-database-detail-page branch April 9, 2026 08:23

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6d118e91bd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +38 to +42
[
showSchemaColumn
? { key: "schema", label: t("common.schema") }
: undefined,
{ key: "name", label: t("common.name") },

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore engine column for non-Postgres table rows

The new React TableMetadataTable builds its column set without an engine field, so non-Postgres databases (where table engine metadata is relevant, e.g. MySQL/MariaDB) no longer show per-table engine values that were previously visible in the legacy table flow. This is a functional regression in the overview tab for those engines and makes it harder to inspect storage-engine-dependent behavior; add a conditional engine column for non-Postgres engines to preserve prior metadata visibility.

Useful? React with 👍 / 👎.

RainbowDashy added a commit that referenced this pull request Apr 9, 2026
* fix(frontend): restore database detail regression parity

* fix(frontend): restore react table detail parity

* fix(frontend): preserve revision import validation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants