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

Fixed a bug that can't open a fileDialog to import/export a SDK file throughout the GUI. #250

Closed
wants to merge 3 commits into from

Conversation

joji2468ng
Copy link
Member

@joji2468ng joji2468ng commented Aug 6, 2023

When export/import_SDKs_action is triggered, it sends False by default to the method, which is why the path argument receives False. The path argument needs to receive None to open a file dialog, To avoid this behavior and ensure None is passed instead, I used a lambda function.

When export/import_SDKs_action is triggered, it sends False by default to the method, which is why the path argument receives False. To avoid this behavior and ensure None is passed instead, I used a lambda function.
@joji2468ng joji2468ng changed the title Fixed a bug that can't import/export a SDK file on the GUI. Fixed a bug that can't open a fileDialog to import/export a SDK file throughout the GUI. Aug 6, 2023
@miquelcampos miquelcampos self-assigned this Aug 6, 2023
@miquelcampos miquelcampos added the bug Something isn't working label Aug 6, 2023
@miquelcampos
Copy link
Member

Hi @joji2468ng thanks for the PR!
I will check it and merge it ASAP 😄

miquelcampos added a commit that referenced this pull request Sep 9, 2023
@miquelcampos
Copy link
Member

@joji2468ng I have copy/pasted the fix of this PR by hand. Since it couldn't merge because of the mistake with the other PR you sent here 😄

Thanks for the contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 4.1.1
Development

Successfully merging this pull request may close these issues.

2 participants