Skip to content

[Feat]: 实现 MySQL Event 管理及编辑查看功能 - #6962

Merged
t8y2 merged 10 commits into
t8y2:mainfrom
onceMisery:feat/issues/4845
Aug 23, 2026
Merged

[Feat]: 实现 MySQL Event 管理及编辑查看功能#6962
t8y2 merged 10 commits into
t8y2:mainfrom
onceMisery:feat/issues/4845

Conversation

@onceMisery

Copy link
Copy Markdown
Contributor

变更说明

  • 增加 MySQL Event 元数据查询与详情接口。
  • 支持 Event 的创建、编辑、删除、刷新和查看。

变更类型

  • 新功能
  • Bug 修复
  • 性能优化
  • 代码重构
  • 文档更新
  • CI / 构建

涉及前端

  • 本 PR 涉及前端改动,已附截图/录屏(见下方)
image image image

验证

  • make check 通过
  • make cargo-check-fast 通过
  • 相关测试通过

关联 Issue

#4845

@onceMisery
onceMisery marked this pull request as draft August 22, 2026 17:22
@github-actions github-actions Bot added area/multiple Touches more than three repository areas db/jdbc Database: JDBC db/mysql Database: MySQL ui-change Changes user-visible interface, text, or visual assets labels Aug 22, 2026
@onceMisery
onceMisery marked this pull request as ready for review August 23, 2026 02:54
@github-actions

Copy link
Copy Markdown
Contributor

🌐 I18n autofill completed

✅ Added translations for 22 new zh-CN i18n keys.

Locale Added entries
es 3
it 3
ja 3
pt-BR 3
zh-TW 3

@t8y2
t8y2 force-pushed the feat/issues/4845 branch 2 times, most recently from 222785f to 5fda266 Compare August 23, 2026 03:42

@t8y2 t8y2 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Maintainer patch applied after rebasing the contribution onto origin/main at 4e09b007c0112d6a733aead75a0f618e66153694.

  • Previous head: ebc0c5d817ec15223905d6488bc5419f590c8332
  • New head: 5fda26670df1a52415a55045f21b4d0dc7a47a5d
  • The only rebase conflict was the pt-BR i18n block: main had added database-rename strings while this PR added MySQL event strings. Both sets were preserved; no feature behavior was discarded.
  • Added positive-integer validation for EVERY interval values in both the frontend SQL builder and Rust SQL builder, with regression cases for empty, zero, non-numeric, and SQL-injection-shaped values. Existing valid units and SQL output remain unchanged.

Validation on the new head: MySQL event Vitest 3/3, pnpm typecheck, cargo test -p dbx-core mysql_event_sql --no-default-features 4 passed, conflict-marker scan, and git diff --check.

@t8y2 t8y2 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Maintainer patch follow-up on the exact head.

  • Previous pushed head: 5fda26670df1a52415a55045f21b4d0dc7a47a5d
  • Current head: 20320b82d0337533e5ec8c6eb2f7799614b6283e
  • Added only the Rust formatting required by the CI rust-fmt-clippy job; no behavior changed.

Validation on the current head: rustfmt --check crates/dbx-core/src/mysql_event_sql.rs, cargo test -p dbx-core mysql_event_sql --no-default-features (4 passed), plus the previously recorded frontend Vitest (3/3), pnpm typecheck, conflict-marker scan, and git diff --check.

@t8y2
t8y2 force-pushed the feat/issues/4845 branch from 82d1ac5 to 8f6d610 Compare August 23, 2026 04:10

@t8y2 t8y2 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Maintainer completion note for the current exact head.

  • Previous pushed head: 82d1ac51b2944c83660660b8996114f1b210fc0c
  • Current head: 8f6d6102343558ca89b32216db73f2131a7e8a19
  • Rebased onto origin/main 701f188e4594fa628af5b17d403389e190939625 without conflicts.
  • The final follow-up only formats the pt-BR translation file required by the frontend CI formatter; behavior is unchanged.

Local validation on this head: frontend oxfmt --check, MySQL event Vitest 3/3, pnpm typecheck, rustfmt --check for the event SQL module, cargo test -p dbx-core mysql_event_sql --no-default-features (4 passed), conflict-marker scan, and git diff --check.

@t8y2 t8y2 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Reviewed exact head 8f6d610; approved for squash merge.

@t8y2
t8y2 merged commit d8bc515 into t8y2:main Aug 23, 2026
15 checks passed
@t8y2

t8y2 commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Thanks for the contribution! Merged in d8bc515, will be released in the next version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/multiple Touches more than three repository areas db/jdbc Database: JDBC db/mysql Database: MySQL ui-change Changes user-visible interface, text, or visual assets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants