-
Notifications
You must be signed in to change notification settings - Fork 98
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
fix: make FilePicker work with latest dialogs #2300
Conversation
7e422db
to
3a2cae5
Compare
Change in .prettierignore is unrelated but necessary after latest workflow update for not having to reformat it after every update. |
@susnux I could track it down that this error only happens when directories are allowed in the picker. When they are disallowed the error is gone, but of course you can't choose a directory for storage then... |
6e31326
to
7d052d0
Compare
Signed-off-by: Christian Hartmann <[email protected]>
7d052d0
to
738731e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Co-authored-by: Ferdinand Thiessen <[email protected]> Signed-off-by: Chartman123 <[email protected]>
This fixes #2298 by adjusting the conditionnal logic in the buttonFactory