Skip to content

Conversation

@jvsena42
Copy link
Member

@jvsena42 jvsena42 commented Aug 14, 2025

FIGMA
Related to #149 , #243

Description

This PR implements the progress cards in the suggestion flow

  • Create Progress Suggestion types
  • Create logic to display
    • Channel force close
    • Chanel coop close
    • Transfer to spending
    • Transfer ready
  • Collect changes in the Home and update the suggestion list
  • Implement radial gradient animation on Suggestion cards
  • Implement logic to replace lightning card with progress cards

Preview

tranfer_rn.mp4
transfer_android.webm

QA Notes

  • Coop close and force closed are not being cached yet. It will be done on an issue related to transfer
  • The navigation will be implemented in other PR

How to reproduce:

  1. Purchase a channel - > navigate to home -> The transfer in progress card should be displayed
  2. Confirm the in-progress transfer -> The lighting ready card should be displayed

@jvsena42 jvsena42 self-assigned this Aug 14, 2025
@jvsena42 jvsena42 marked this pull request as draft August 14, 2025 12:50
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

detekt found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@jvsena42 jvsena42 marked this pull request as ready for review August 18, 2025 15:25
@jvsena42 jvsena42 requested a review from ovitrif August 18, 2025 16:30
@jvsena42 jvsena42 enabled auto-merge August 18, 2025 16:32
Copy link
Collaborator

@ovitrif ovitrif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT 🎉

Tested

  • new transfer to spending + mining & watching the card update on the Home screen.

Added comments for some cleanup suggestions but nothing blocking.

Mainly the comment about the e2e test id Suggestion-lightningSettingUp should be addresses ASAP in a new PR.

@jvsena42 jvsena42 merged commit e84dc69 into master Aug 19, 2025
5 checks passed
@jvsena42 jvsena42 deleted the feat/card-pending-transactions branch August 19, 2025 09:47
@jvsena42 jvsena42 mentioned this pull request Aug 19, 2025
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.

3 participants