Skip to content
This repository has been archived by the owner on Aug 8, 2024. It is now read-only.

fix: Fixed chat crash bug #687

Merged
merged 2 commits into from
Sep 15, 2023
Merged

fix: Fixed chat crash bug #687

merged 2 commits into from
Sep 15, 2023

Conversation

ENORMOUZ
Copy link
Contributor

Fixed a bug when trying to delete chat lines
I lost 50stx to this bug :(

@ENORMOUZ ENORMOUZ requested a review from a team as a code owner August 22, 2023 01:49
@magicus
Copy link
Member

magicus commented Aug 24, 2023

Ooh, that is such sadness! But how did this bug cause you to lose emeralds? 🤔

@ENORMOUZ
Copy link
Contributor Author

Crashed game while transferring items, which caused the item to drop and despawn :(

@magicus
Copy link
Member

magicus commented Aug 24, 2023

Oh. :( I am sorry.

Copy link
Member

@ryanzhoudev ryanzhoudev left a comment

Choose a reason for hiding this comment

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

Kind of not good catching all exception but it's legacy...

@ENORMOUZ
Copy link
Contributor Author

Can this be merged?

@ryanzhoudev ryanzhoudev merged commit 76ed2da into Wynntils:development Sep 15, 2023
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants