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

Fixes by Goose Action #8

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

Conversation

github-actions[bot]
Copy link

This is an auto-generated PR with fixes by Goose.
Task: Update SendPage to remove "Verifying DAP..." loading message

similar to what was achieved in TBD54566975#281, we want to consolidate any consecutive loading spinners into one loading page
currently, you can go to SendPage via the send tab in the bottom navigation bar. from here, typing in a dap (e.g. @moegrammer/didpay.me) and hitting the next button triggers a "Verifying DAP..." loading message followed by a "Fetching offerings..." loading message which means you have entered the payment flow starting at PaymentAmountPage
the goal of this issue is to remove the "Verifying DAP..." loading message in SendPage and instead put a small loading spinner to the right of the DAP input form when resolving the DAP
old flow: input DAP -> hit next -> see "Verifying DAP..." -> see "Fetching offerings..." -> enter PaymentAmountPage
new flow: input DAP -> hit next -> see small loading spinner next to DAP input form -> see "Fetching offerings..." -> enter PaymentAmountPage

Link to issue

@github-actions github-actions bot added automated pr goose Ask goose to fix it labels Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated pr goose Ask goose to fix it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant