Skip to content

[engine] Gift card transactions ledger minds#4165

Ben requested to merge feat/gift-card-txs-m4165 into master

Ticket(s) / Related Merge Requests

Blocked by !1396 (merged) and its front end

Summary of Changes

Add gift card transactions ledger to the wallet.

Testing Considerations

Recommend using an account with gift cards for this - feel free to use playwright_tests_user but please do not spend any of the gift cards or you may break E2E tests.

  • Test the credits option in the wallet section shows you your credits.
  • Test the status filters - after selecting one query string should update - on refresh it should load the appropriate feed.
  • Test that clicking on a gift card OR to view transactions take you to the gift cards transactions - please DO NOT USE PLAYWRIGHT TESTS USER TO SPEND CREDITS.
  • Check balance in summary works.
  • Test feat flag.
  • Deviations from design have been agreed with @mwrobs.
  • Test mobile viewport works reasonably (no given designs to reference but ran what I did past design).
  • Check that token price is hidden on all pages but the token page AND on mobile viewports too small to support the 3 tabs and the price badge.
  • Test that tab storage guard works - when you are on the credits tab, navigate away then back, you should be returned to the credits tab.
  • Test that clicking usernames and links in the storage tab work.

Note: it is known that the integration tests are broken right now - this change does not impact any integration tested routes.

Deployment Considerations

Standard deployment but note that it is behind a the gift card claim code feature flag.

Regression Scope

Wallet, new gift card transaction section.

Platform Affected (web, mobile, etc)

Developer Testing Completed

Screenshots / Screen Recording

simplescreenrecorder-2023-07-24_10.09.51

Feat flag off

image

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Ben

Merge request reports