Skip to content

style: add success message after user clicks "Send Funds"#13100

Merged
rjan90 merged 4 commits intofilecoin-project:masterfrom
barbaraperic:bp/ux-enhacement
May 19, 2025
Merged

style: add success message after user clicks "Send Funds"#13100
rjan90 merged 4 commits intofilecoin-project:masterfrom
barbaraperic:bp/ux-enhacement

Conversation

@barbaraperic
Copy link
Contributor

@barbaraperic barbaraperic commented May 7, 2025

Related Issues

Closes #13064

This PR improves user experience by adding a success message after funds are sent.

Proposed Changes

  • Add a new success page template that shows after successful fund transfers
  • Include transaction CID in the success message for user reference
  • Enhance user flow by providing clear confirmation of successful transactions
  • Improve overall UX feedback when users interact with the Lotus Fountain

Additional Info

CleanShot.2025-05-07.at.10.47.46.mp4

Checklist

Before you mark the PR ready for review, please make sure that:

@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz May 7, 2025
@rjan90
Copy link
Contributor

rjan90 commented May 7, 2025

Thanks for the PR @barbaraperic. It looks like there is a conflict that needs to be resolved /lotus-fountain/site/main.css

@rjan90 rjan90 self-assigned this May 7, 2025
@rjan90 rjan90 self-requested a review May 7, 2025 15:03
@rjan90 rjan90 moved this from 📌 Triage to 🔎 Awaiting Review in FilOz May 7, 2025
@rjan90
Copy link
Contributor

rjan90 commented May 9, 2025

@barbaraperic Do you want to resolve the conflicts? Or are you okay with me sorting those out?

@barbaraperic
Copy link
Contributor Author

barbaraperic commented May 12, 2025

@barbaraperic Do you want to resolve the conflicts? Or are you okay with me sorting those out?

Feel free to sort them out, let me know if I can help :)

@rjan90 rjan90 added the skip/changelog This change does not require CHANGELOG.md update label May 19, 2025
barbaraperic and others added 2 commits May 19, 2025 09:38
Co-authored-by: Phi-rjan <orjan.roren@gmail.com>
fix: remove duplicate header and handle datacap response correctly
@github-project-automation github-project-automation bot moved this from 🔎 Awaiting Review to ✔️ Approved by reviewer in FilOz May 19, 2025
@rjan90
Copy link
Contributor

rjan90 commented May 19, 2025

Merging event with the one failure check, as it has been captured here that the itest-eth_api_f3 is consistently failing now: #13116 (comment)

@rjan90 rjan90 merged commit 3d46128 into filecoin-project:master May 19, 2025
94 of 95 checks passed
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz May 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

[Lotus Fountain Faucet] Successful Request Feedback missing

2 participants