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.

Sign up for GitHub

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

Jump to bottom

Fix delete confirmation bug #69

Closed

JustinDSN wants to merge 1 commit into sergdort:master from JustinDSN:bugfix/61-delete-confirm

Contributor

JustinDSN commented May 16, 2019 •

edited

Loading

Builds upon the solution in PR #63 to fix bug #61.

Changes confirm dialog from Void to Bool observable
True represents Yes, False represents No
Dismisses the dialog if disposed.


          Fix delete confirmation bug

Contributor Author

JustinDSN commented Jun 3, 2019

@sergdort can we get this PR reviewed and merged?

Contributor Author

JustinDSN commented May 31, 2020 •

edited

Loading

@sergdort are you interested in this patch? - I can fix the merge conflicts or close it.

JustinDSN closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet