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: 🐛 log stack only saved for logged_messages.all #278

Merged
merged 1 commit into from
Jun 17, 2023

Conversation

KANAjetzt
Copy link
Member

Previously, the same log entry was only saved to the previous entry in ModLoaderStore.logged_messages.all[logMD5]. This change adds a check for an existing_entry and adds to the entry stacks in ModLoaderStore.logged_messages.by_mod and ModLoaderStore.logged_messages.by_type.

Previously, the same log entry was only saved to the previous entry in `ModLoaderStore.logged_messages.all[logMD5]`. This change adds a check for an `existing_entry` and adds to the entry stacks in `ModLoaderStore.logged_messages.by_mod` and `ModLoaderStore.logged_messages.by_type`.
@KANAjetzt KANAjetzt added the bug Something isn't working label Jun 16, 2023
@KANAjetzt KANAjetzt added this to the v6.0.0 milestone Jun 16, 2023
@KANAjetzt KANAjetzt requested review from ithinkandicode, Qubus0, otDan and a team June 16, 2023 08:07
@KANAjetzt KANAjetzt self-assigned this Jun 16, 2023
@KANAjetzt KANAjetzt mentioned this pull request Jun 16, 2023
15 tasks
Copy link
Collaborator

@Qubus0 Qubus0 left a comment

Choose a reason for hiding this comment

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

good catch

@KANAjetzt KANAjetzt added this pull request to the merge queue Jun 17, 2023
Merged via the queue into GodotModding:development with commit 90334fc Jun 17, 2023
@KANAjetzt KANAjetzt deleted the fix_store_log branch June 19, 2023 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants