Skip to content
This repository has been archived by the owner on Jun 16, 2022. It is now read-only.

LL-2227 - Fix - Operation details - Fix navigation with account page #2482

Conversation

nparigi-ledger
Copy link
Contributor

After navigating to the operation's account page, when clicking "back" button you should go back to operation and not go to Accounts page

Type

Bug fix

Context

https://ledgerhq.atlassian.net/browse/LIVE-2227

Parts of the app affected / Test plan

Operation details

…ount page, when clicking "back" button you should go back to operation and not go to Accounts page
@nparigi-ledger nparigi-ledger requested a review from a team as a code owner May 4, 2022 13:24
@github-actions
Copy link

github-actions bot commented May 4, 2022

Thanks for your contribution.
To be able to merge in develop branch, you need to:

  • pass the CI
  • if needed, run /generate-screenshots
  • have a dev review
  • have a QA review
  • if needed, /upgrade-llc

Why /generate-screenshots ?

If your PR contains UI related changes,
it might be necessary to regenerate screenshots.

Why /upgrade-llc ?

If your PR requires an update to the ledger-live-common library,
once the PR is merged on develop on ledger-live-common side,
you need to run /upgrade-llc to switch back to ledger-live-common@develop here before merging.

@LFBarreto LFBarreto merged commit 79ffe07 into develop May 4, 2022
@LFBarreto LFBarreto deleted the bugfix/LIVE-2227-operation-details-account-page-redirect-back branch May 4, 2022 14:09
Copy link

@ychen-ledger ychen-ledger left a comment

Choose a reason for hiding this comment

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

Validated with:

  • LLM 3.0.8
  • Pixel 4 Android 12/ iPhone 11 iOS 14.7

@valpinkman valpinkman mentioned this pull request May 5, 2022
@lambertkevin lambertkevin mentioned this pull request May 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants