Skip to content

Conversation

@ch0m5
Copy link
Contributor

@ch0m5 ch0m5 commented Jan 17, 2025

Addresses #852.

Note that this PR changes the "options" Array passed to the Godot Scene from essentially Array[String] to Array[Dictionary], where each Dictionary is a data-package for each Choice, rather than just its text (the prior case).

Therefore, it changes how Choice data is sent and interpreted from the GDExtension to GDScript, therefore affecting compatibility with any Orchestrations with Dialogue created prior to this PR.

- All choices are sent to the Scene, not just visible ones
- Choices is no longer an Array, but a Dictionary with "visible", "text", and "metadata" entries
- Included "metadata" entry for DialogueMessage data as well as for each choice's Dictionary
@Naros Naros linked an issue Jan 31, 2025 that may be closed by this pull request
Carles Homs added 2 commits February 5, 2025 13:00
- All choices are sent to the Scene, not just visible ones
- Choices is no longer an Array, but a Dictionary with "visible", "text", and "metadata" entries
- Included "metadata" entry for DialogueMessage data as well as for each choice's Dictionary
@Naros
Copy link
Member

Naros commented Feb 11, 2025

Hi @ch0m5, could you rebase your changes against main rather than doing a merge commit

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.

Add Node Metadata Access to DialogueMessage and DialogueChoice Nodes

2 participants