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.

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

fix(mm): correctly handle lobby not found error if joining direclty to lobby id that doesn't exist #867

Conversation

NathanFlurry
Copy link
Member

Changes

Copy link
Contributor

graphite-app bot commented Jun 6, 2024

Your org requires the Graphite merge queue for merging into main

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

You can enable merging using labels in your Graphite merge queue settings.

Copy link
Contributor

graphite-app bot commented Jun 6, 2024

Merge activity

…o lobby id that doesn't exist (#867)

<!-- Please make sure there is an issue that this PR is correlated to. -->

## Changes

<!-- If there are frontend changes, please include screenshots. -->
@NathanFlurry NathanFlurry force-pushed the 06-05-fix_mm_call_mm-lobby-cleanup_from_mm-gc_even_for_preemptive_lobbies_without_sql_row branch from c9aa4c5 to 5315a9a Compare June 6, 2024 17:28
@NathanFlurry NathanFlurry force-pushed the 06-05-fix_mm_correctly_handle_lobby_not_found_error_if_joining_direclty_to_lobby_id_that_doesn_t_exist branch from 24fdc37 to af3513a Compare June 6, 2024 17:29
@NathanFlurry NathanFlurry changed the base branch from 06-05-fix_mm_call_mm-lobby-cleanup_from_mm-gc_even_for_preemptive_lobbies_without_sql_row to main June 6, 2024 17:32
@graphite-app graphite-app bot merged commit af3513a into main Jun 6, 2024
6 of 10 checks passed
@graphite-app graphite-app bot deleted the 06-05-fix_mm_correctly_handle_lobby_not_found_error_if_joining_direclty_to_lobby_id_that_doesn_t_exist branch June 6, 2024 17:33
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