Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

JAMES-3513 Wrong UID dispatched on the EventBus for StoreMessageIdMan… #325

Closed
wants to merge 1 commit into from
Closed

JAMES-3513 Wrong UID dispatched on the EventBus for StoreMessageIdMan… #325

wants to merge 1 commit into from

Conversation

chibenwa
Copy link
Contributor

@chibenwa chibenwa commented Mar 8, 2021

…ager::setInMailboxes

Given msg1 in mbx1 with uid 12 and in mbx2 with uid 58
When I set it in mbx3
Then two deletion events are dispatched, one in mbx1 with uid 12 ok, one in mbx2 and uid 12.

@chibenwa
JAMES-3513 Wrong UID dispatched on the EventBus for StoreMessageIdMan…
c06035b 
…ager::setInMailboxes

```
Given msg1 in mbx1 with uid 12 and in mbx2 with uid 58
When I set it in mbx3
Then two deletion events are dispatched, one in mbx1 with uid 12 ok, one in mbx2 and uid 12.
```
Arsnael
Arsnael approved these changes Mar 8, 2021
View reviewed changes
Copy link
Contributor

@Arsnael Arsnael left a comment

Choose a reason for hiding this comment

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

Nice catch!

@chibenwa
Copy link
Contributor Author

Merged

@chibenwa chibenwa closed this Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Arsnael Arsnael Arsnael approved these changes

@LanKhuat LanKhuat LanKhuat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@chibenwa @Arsnael @LanKhuat