Skip to content

[WPB-8892] Add block list operations to the user subsystem#4167

Merged
mdimjasevic merged 4 commits intodevelopfrom
WPB-8892-polysemy-user-subsystem-add-block-list-operations
Jul 24, 2024
Merged

[WPB-8892] Add block list operations to the user subsystem#4167
mdimjasevic merged 4 commits intodevelopfrom
WPB-8892-polysemy-user-subsystem-add-block-list-operations

Conversation

@battermann
Copy link
Contributor

@battermann battermann commented Jul 23, 2024

Refactor the code in Brig by moving the email block-listing actions into a separate effect in the wire-subsystems library.

Tracked by https://wearezeta.atlassian.net/browse/WPB-8892.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@echoes-hq echoes-hq bot added the echoes: technical-roadmap/throughput More specific category, to highlight task aiming at improving the development velocity and effici... label Jul 23, 2024
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jul 23, 2024
@mdimjasevic mdimjasevic force-pushed the WPB-8892-polysemy-user-subsystem-add-block-list-operations branch from debe27b to b3e3e22 Compare July 24, 2024 08:57
@mdimjasevic mdimjasevic changed the title WPB-8892 Add block list operations [WPB-8892] Add block list operations to the user subsystem Jul 24, 2024
@mdimjasevic mdimjasevic marked this pull request as ready for review July 24, 2024 09:26
Copy link
Contributor

@mdimjasevic mdimjasevic left a comment

Choose a reason for hiding this comment

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

Looks good! I should get more familiar with the concept of mini-backend and how the interpreter for the block-listing effect fits into this idea.

@mdimjasevic mdimjasevic merged commit 2259719 into develop Jul 24, 2024
@mdimjasevic mdimjasevic deleted the WPB-8892-polysemy-user-subsystem-add-block-list-operations branch July 24, 2024 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: technical-roadmap/throughput More specific category, to highlight task aiming at improving the development velocity and effici... ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants