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

Various minor bugfixes #2817

Merged
merged 7 commits into from
Jun 29, 2022
Merged

Various minor bugfixes #2817

merged 7 commits into from
Jun 29, 2022

Conversation

Ghabry
Copy link
Member

@Ghabry Ghabry commented Jun 25, 2022

This fixes various bugs encountered over the last weeks that are almost all one-line fixes.

  • record-input was found by makodeoz on Discord
  • facing by @TrunXGIT (please retest!)
  • the others were found by Jan (Deep 8) and are backported from the Deep8 fork

…f stale event data

Otherwise the current interpreter continues after loading when the savegame has no interpreter running
This was already fixed earlier properly making this workaround obsolete
This is currently not possible to do.
Found it in Deep8 which has a dynrpg command for translation change.
@fdelapena fdelapena added Savegames RPG_RT Patches Emscripten WebAssembly/JavaScript port for web browsers labels Jun 25, 2022
@TrunXGIT
Copy link

Tested the build and at least the 3 cases in the sample project behave the same now in Player.exe and RPG_RT.exe.
Good job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

4 participants