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

1.x converter "jump" working by prep loop func + character "positions" debugged #2009

Merged
merged 6 commits into from Jan 20, 2024

Conversation

metanoder
Copy link
Contributor

@metanoder metanoder commented Jan 10, 2024

Tested the dialogic-1 converter today. Noticed the change_timeline() to jump conversion needed touching up.

Added a prep loop to populate a dictionary with timeline keys and file names to be able to populate the converted timeline to write out proper "jump nextTimeline/" in editor.

*Noticed the character position conversion for 2.0 now mirrors 1.x again. Deleted the +1 in the code.

**Committed a fix for conversion for an empty if-statement definition. #1494

@metanoder metanoder changed the title dialogic-1 converter change_timeline -> jump fix by prep loop func 1.x converter "jump" working by prep loop func + character "positions" debugged Jan 11, 2024
@metanoder
Copy link
Contributor Author

Committed a fix for conversion for an empty if-statement definition. #1494

Checks if there's a definition before proceeding to look for path and name. Won't break the conversion if there's an empty if-statement definition, and proceed with the conversion, while outputting there's a problem in the converter dialog box.

@Jowan-Spooner
Copy link
Collaborator

@metanoder Thanks for your PR. Currently I have no time to test the converter or your PR. Unfortunately @exelia-antonov who was mainly responsible for the converter has very little time these days as well. Thus, I'm going to merge this without testing.

The current plan is to remove the converter from dialogic core before we go beta and just put a link to another repo hosting it.
If you are interested in helping keeping the converter working/alive, feel free to join us in the #development channel on emilios discord, I'm sure we'd all be happy if someone wanted to work on it.

@Jowan-Spooner Jowan-Spooner merged commit 730b4d7 into dialogic-godot:main Jan 20, 2024
2 checks passed
Invertex pushed a commit to Invertex/dialogic that referenced this pull request Jan 26, 2024
…" debugged (dialogic-godot#2009)

* dialogic-1 converter change_timeline -> jump fix by prep loop func

* 1.x vs 2.x character positions now match, converter updated

* fixed empty if-statement conversion break
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.

None yet

2 participants