Skip to content

fix(migrations): remove redundant enabled column from projects table#44

Merged
linkai0924 merged 1 commit intomainfrom
fix/migrations
Apr 7, 2026
Merged

fix(migrations): remove redundant enabled column from projects table#44
linkai0924 merged 1 commit intomainfrom
fix/migrations

Conversation

@linkai0924
Copy link
Copy Markdown
Collaborator

@linkai0924 linkai0924 commented Apr 7, 2026

Remove the 'enabled' boolean column and its index from the projects table

migration since project status is now tracked via 'enabled_at' timestamp.

Summary by CodeRabbit

  • Chores
    • Updated portal submodule to latest version.
    • Refined project management database schema by removing an unused field from the projects table.

Remove the 'enabled' boolean column and its index from the projects table

migration since project status is now tracked via 'enabled_at' timestamp.

Signed-off-by: 林凯90331 <90331@sangfor.com>

Co-authored-by: CoStrict <zgsm@sangfor.com.cn>
@linkai0924 linkai0924 merged commit 545011c into main Apr 7, 2026
4 of 7 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 7, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6d666ed8-e403-4341-925f-1e7a468994e9

📥 Commits

Reviewing files that changed from the base of the PR and between f8979b0 and 5a1d81f.

📒 Files selected for processing (2)
  • portal
  • server/migrations/20260401173000_create_project_management_tables.sql

📝 Walkthrough

Walkthrough

Two updates to the codebase: the portal submodule pointer is advanced to a new commit, and the enabled column along with its index is removed from the projects table in a database migration.

Changes

Cohort / File(s) Summary
Submodule Update
portal
Updated submodule pointer to commit 658d54afc0ee1e27456895bdc2402c36e712e674.
Database Schema Migration
server/migrations/20260401173000_create_project_management_tables.sql
Removed enabled column and its index idx_projects_enabled from the projects table. Note: Down migration still references dropping idx_projects_enabled, which may indicate a migration consistency issue.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A hop through the code, a commit so new,
The portal spins onward, a submodule's view.
Columns take flight when their time has expired,
The enabled field's gone—a schema retired.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/migrations

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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