Skip to content

Move controller messenger type demonstration to its own test suite - #565

Merged
rekmarks merged 2 commits into
mainfrom
move-messenger-type-demonstration
Aug 27, 2021
Merged

Move controller messenger type demonstration to its own test suite#565
rekmarks merged 2 commits into
mainfrom
move-messenger-type-demonstration

Conversation

@rekmarks

Copy link
Copy Markdown
Member

Previously, we demonstrated how controller messenger generics could be a union of actions and events from different controllers in the currency rate controller tests. Although we have use cases in production that demonstrate this, I think it's worthwhile to keep an example in the tests. This PR moves this type demonstration into the controller messenger test suite.

@rekmarks
rekmarks requested a review from GudahttAugust 21, 2021 19:28
@rekmarks
rekmarks requested a review from a team as a code ownerAugust 21, 2021 19:28

@GudahttGudahtt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@rekmarks
rekmarks merged commit 2cdbdc4 into mainAug 27, 2021
@rekmarks
rekmarks deleted the move-messenger-type-demonstration branch August 27, 2021 21:22
@adonesky1adonesky1 mentioned this pull request Aug 27, 2021
Merged
amitabh94 pushed a commit to amitabh94/core that referenced this pull request Jun 16, 2026
* 22.4.0
* chore(release/22.4.0): update changelog
---------
Co-authored-by: github-actions <github-actions@github.com>
Co-authored-by: rarquevaux <r.arquevaux@gmail.com>
Sign up for freeto 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.

2 participants

@rekmarks@Gudahtt