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.

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

feat: Wallet tx simple view #1787

Draft
wants to merge 146 commits into
base: master
Choose a base branch
from
Draft

feat: Wallet tx simple view #1787

wants to merge 146 commits into from

Conversation

nelitow
Copy link
Contributor

@nelitow nelitow commented Jan 20, 2025 •
edited
Loading

Closes: FE-1310

Wallet Tx Simple View

  • Tx history
  • Tx approval
image image

Checklist

  • I've added error handling for all actions/requests.
  • I've reviewed how my changes will display in the UI.
  • I've checked all the copy (text) changes or additions in this PR.
  • I've included references to the issues being closed on GitHub and/or Linear.
  • I've ensured that the documentation is up to date and reflects any changes.
  • I've added documentation links where it may be helpful.
  • I reviewed the entire PR myself.

nelitow added 14 commits January 20, 2025 16:03
@nelitow
feat: grouped wallet view WIP
109794e
@nelitow
feat: separate into components
5c95579
@nelitow
fix(ecosystem): enhance error handling in fetchProjects method
bf478f6
@nelitow
feat(Transaction): add totalAmount and operationCount to ContractCall…
2975d8a 
…Metadata and SwapMetadata types
@nelitow
refactor(TxViewSimpleWrapper): remove console logs and simplify loadi…
a8f965d 
…ng logic
@nelitow
refactor(TxFeeSimple): streamline fee display logic and enhance asset…
535df29 
… handling
@nelitow
refactor(TxOperationCard): remove console logs and simplify group det…
2017132 
…ection logic
@nelitow
refactor(TxOperationContent): remove console logs to clean up code
6a06bdf
@nelitow
feat(TxOperationHeader): enhance contract call display with project info
3deb369
@nelitow
feat(TxOperationSend): enhance transaction display with asset amounts…
9ddd6cf 
… and contract call indication
@nelitow
feat(TxOperationSwap): enhance swap operation display with ecosystem …
602c0cc 
…project info and loading state
@nelitow
feat(useEcosystemProject): implement custom hook for fetching and cac…
c28a430 
…hing ecosystem project data
@nelitow
refactor(simplifyTransaction): improve grouping logic for transaction…
a23a780 
… operations by including asset ID for SEND operations and enhancing contract call grouping
@nelitow
Merge branch 'master' into nj/feat/tx-simple-view
e8af0e8
@nelitow
refactor(TxFeeSimple): simplify component structure by removing Card …
05aa994 
…wrapper and enhancing layout with Box.Stack for improved readability
@nelitow
feat(TxOperationSend): enhance transaction display by adding account …
f3009c6 
…name resolution, avatar support, and improved asset formatting
@nelitow
refactor(TxOperationSwap): remove redundant address display and copy …
b3492e3 
…functionality for cleaner UI
@github-actions GitHub Actions
Copy link
Contributor

❌ Missing required references: " + missingRefs.join(" and ") + ". Please include both a GitHub and a Linear issue reference in your PR description.

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

@luizstacio luizstacio Awaiting requested review from luizstacio luizstacio will be requested when the pull request is marked ready for review luizstacio is a code owner

@LuizAsFight LuizAsFight Awaiting requested review from LuizAsFight LuizAsFight will be requested when the pull request is marked ready for review LuizAsFight is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@nelitow nelitow

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nelitow