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

[AVW-2908] Transaction Review Header #1079

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Conversation

kugel3
Copy link
Contributor

@kugel3 kugel3 commented Apr 9, 2024

Jira ticket: ABW-2908

Description

Show the title in the navigation bar in Transaction Review, when scrolling away the other title.

Notes

We could do the same thing in other places too, like the home screen, it's fairly painless.

How to test

  • Go to transaction view and scroll down far enough for the main title to disappear.

Screenshot

Video

TxReview.mov

PR submission checklist

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

@kugel3 kugel3 changed the title [AVW-2908] review transaction header [AVW-2908] Transaction Review Header Apr 9, 2024
@GhenadieVP
Copy link
Contributor

How does it look for incoming transactions from a dApp? it should be like this

@kugel3
Copy link
Contributor Author

kugel3 commented Apr 9, 2024

Well it doesn't look like that, I am just using the navigationTitle property. Will try to use a toolbar item instead

@kugel3
Copy link
Contributor Author

kugel3 commented Apr 9, 2024

Now it looks like that...

Copy link
Contributor

@GhenadieVP GhenadieVP left a comment

Choose a reason for hiding this comment

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

Niice, just check if the changes in Package.resolved are intentional.

@@ -1,5 +1,4 @@
{
"originHash" : "3313f884630244f405efa4cdbbcb2486d097b16289931114ea3c6826384f24b9",
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be removed?

@@ -15,8 +14,8 @@
"kind" : "remoteSourceControl",
"location" : "https://github.com/AppsFlyerSDK/AppsFlyerFramework-Strict",
"state" : {
"revision" : "a22ea3af34d4631123bd54f55f95f7b7a9cd905f",
"version" : "6.13.2"
"revision" : "f9e6a597252ab719805033a7912c64d2dd1f077e",
Copy link
Contributor

Choose a reason for hiding this comment

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

intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no

@kugel3 kugel3 force-pushed the ABW-2908_Review-Transaction-header branch from 219f3a4 to e1b8e85 Compare April 10, 2024 11:19
@kugel3 kugel3 merged commit 62608f3 into main Apr 10, 2024
6 checks passed
@kugel3 kugel3 deleted the ABW-2908_Review-Transaction-header branch April 10, 2024 12:33
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