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

export asset list button #121

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

senos06
Copy link

@senos06 senos06 commented Jun 9, 2023

@biast12
Copy link
Contributor

biast12 commented Jun 9, 2023

there's a few things that doesn't work in this pr, like it's crashing if you click the button without having anything loaded, and it exports it in the LegionPlus.exe dir and not under exported_files/lists as other assets and lists does

also a thing that would be really nice and easy to add would be having it being named the .rpak files names if you only got one loaded, just like the list flag does rn, the code is at the bottom in ExportManager.cpp

@biast12
Copy link
Contributor

biast12 commented Jun 9, 2023

i would recommend using ExportDirectory as the dir path, that would both somewhat fix the exporting in the LegionPlus.exe dir and do so the ExportBrowseFolder setting works for the lists button too

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