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

[ABW-2589] Transaction History Part Deux #1052

Merged
merged 58 commits into from
Mar 22, 2024

Conversation

kugel3
Copy link
Contributor

@kugel3 kugel3 commented Mar 18, 2024

Jira ticket: paste link here

Complements / depends on / relates to #link to other PR

Description

This PR solves / fixes / adds / includes functionality 1 and functionality 2.

Notes

Additional notes.

How to test

Happy path (or test variation 1)

  1. Step 1
  2. Step 2
  3. Verify

Unhappy path (or test variation 2)

  1. Step 1
  2. Step 2
  3. Verify

Screenshot

Screen 1 Before Screen 1 After
paste link here paste link here

Video

paste link here

PR submission checklist

  • I have tested account to account transfer flow and have confirmed that it works

@kugel3 kugel3 force-pushed the ABW-2589_Transaction-history-part-deux branch 4 times, most recently from 4fe4888 to f09bb4b Compare March 22, 2024 12:43
@kugel3 kugel3 force-pushed the ABW-2589_Transaction-history-part-deux branch from daa0daf to 0dfa306 Compare March 22, 2024 13:16
@@ -55,7 +55,7 @@ extension TransactionReviewAccount.State {
extension TransactionReviewAccount {
public struct ViewState: Equatable {
let account: TransactionReview.Account
let transfers: [ResourceBalance] // FIXME: GK use viewstate?
let transfers: [TransactionReview.Transfer] // FIXME: GK use viewstate?
Copy link
Contributor

Choose a reason for hiding this comment

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

Goal Keeper?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gustaf Kugelberg

@GhenadieVP GhenadieVP merged commit 9634ecf into main Mar 22, 2024
6 checks passed
@GhenadieVP GhenadieVP deleted the ABW-2589_Transaction-history-part-deux branch March 22, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants