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

fix(fb): display error alert in modal #776

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

barrytra
Copy link
Contributor

@barrytra barrytra commented Jun 3, 2024

fixes #775
The alert message has been shifted inside modal.
However I was not able to produce the error locally, so its requested to produce the error locally while reviewing it. and please let know if it works fine.
PS: If anyone gets to know how to produce error, I would love to learn it as well.

@theborakompanioni
Copy link
Collaborator

theborakompanioni commented Jun 4, 2024

PS: If anyone gets to know how to produce error, I would love to learn it as well.

e.g. this gets triggered when the wallet is not "Fidelity Bond"-enabled, when the wallet is created via the CLI and the users presses n on the question Would you like this wallet to support fidelity bonds? (https://jamdocs.org/FAQ/#how-to-import-an-existing-wallet-via-the-command-line).

We could load the timelocked address immediately after selecting the date, so the error would get triggered earlier in the process. Currently the user selects a UTXO and only then the address is loaded.. maybe this can be considered in #650?

PR looks good! Approved 🚀

@theborakompanioni theborakompanioni changed the title (fix/fb):alert inside modal fix(fb): display error alert in modal Jun 4, 2024
@barrytra barrytra merged commit fef6260 into joinmarket-webui:master Jun 4, 2024
3 checks passed
@barrytra barrytra deleted the fix/error branch June 4, 2024 08:25
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.

(FB) : error Alert is displayed in the background of modal.
2 participants