Skip to content

[_]: refactor(account): remove network_bucket_id from mail accounts - #113

Open
jzunigax2 wants to merge 1 commit into
feat/purge-suspended-accountsfrom
fix/remove-left-over-code
Open

[_]: refactor(account): remove network_bucket_id from mail accounts#113
jzunigax2 wants to merge 1 commit into
feat/purge-suspended-accountsfrom
fix/remove-left-over-code

Conversation

@jzunigax2

Copy link
Copy Markdown
Contributor
  • Created a migration to drop the network_bucket_id column from the mail_accounts table.
  • Updated MailAccount interface and model to remove references to networkBucketId.
  • Removed related methods and tests for setting networkBucketId in AccountRepository and AccountRepository tests.
  • Adjusted fixtures to reflect the removal of networkBucketId.

- Created a migration to drop the network_bucket_id column from the mail_accounts table.
- Updated MailAccount interface and model to remove references to networkBucketId.
- Removed related methods and tests for setting networkBucketId in AccountRepository and AccountRepository tests.
- Adjusted fixtures to reflect the removal of networkBucketId.
@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 4015ca22-639d-4fe3-b8d5-3e768edfdc4a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@jzunigax2
jzunigax2 requested a review from xabg2 August 21, 2026 20:47
@jzunigax2 jzunigax2 self-assigned this Aug 21, 2026
@jzunigax2 jzunigax2 added the migration-required A migration is included in this PR label Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

migration-required A migration is included in this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants