Skip to content

Conversation

@Yanis002
Copy link
Collaborator

@Yanis002 Yanis002 commented Dec 2, 2024

The quick import file was a bit messy so I decided to improve it a bit in the process, also I wanted to add the missing versions to the enum

Now you can use the quick import feature to import data from overlays, I tested it with D_808BCE20


I decided to add a new preview setting to ignore the cutscene misc command that stops the cutscene, in some cases it can be useful

@Yanis002 Yanis002 added oot Has to do with the Ocarina of Time 64 side codebase Code maintenance/cleanup labels Dec 2, 2024
@Yanis002 Yanis002 changed the title Add missing versions to oot_versions_items and add overlay support for quick import [OoT] Add missing versions to oot_versions_items and add overlay support for quick import Dec 2, 2024
Copy link
Collaborator

@Lilaa3 Lilaa3 left a comment

Choose a reason for hiding this comment

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

Some questions before I approve

Comment on lines +209 to +210
if len(parsedCutscenes) == 0:
raise PluginError("ERROR: No cutscene was found.")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this intended for scene imports as well? Deku tree inside won´t import with Cutscene importing on now.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll investigate that, it should import properly so there might be something going on

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I tested and it works properly, though it won't work if you already imported that cutscene before, which might be a concern so I changed it to make it print a warning instead of skipping the import entirely

@Dragorn421 Dragorn421 added the merge soon Will be merged in a few days at most if nothing else comes up label May 19, 2025
@Dragorn421 Dragorn421 merged commit 8336f42 into Fast-64:main May 20, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codebase Code maintenance/cleanup merge soon Will be merged in a few days at most if nothing else comes up oot Has to do with the Ocarina of Time 64 side

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants