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: message loading #6210

Merged
merged 3 commits into from
Jun 13, 2022
Merged

fix: message loading #6210

merged 3 commits into from
Jun 13, 2022

Conversation

BeksOmega
Copy link
Collaborator

Patch containing #6184 and rebuilt message files.

* fix: loading messages in the browser

* chore: fix comment

* fix: change unwrapped message files to write to a new object, rather than Blockly.Msg

* fix: fixup exports

* fix: PR comments

* fix: change to use for-in loop

* fix: ES6 compatibility and formatting
@BeksOmega BeksOmega requested a review from maribethb June 13, 2022 21:29
@BeksOmega BeksOmega requested a review from a team as a code owner June 13, 2022 21:29
@BeksOmega BeksOmega merged commit e1bfbfa into google:master Jun 13, 2022
@BeksOmega BeksOmega mentioned this pull request Jun 16, 2022
4 tasks
@maribethb
Copy link
Contributor

@BeksOmega The rebuilt files were never updated in develop, if you have time could you make this change in develop as well? It makes master and develop different in confusing ways (develop is behind master)

@BeksOmega BeksOmega mentioned this pull request Aug 3, 2022
4 tasks
@BeksOmega BeksOmega deleted the rc/8.0.4 branch May 14, 2024 16:26
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.

2 participants