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

Edit transaction fee screen for EIP-1559 and legacy transactions #1943

Merged
merged 8 commits into from
Jun 21, 2023

Conversation

elgatovital
Copy link
Contributor

Handles #1932

Changes proposed in this pull request:

  • Add fragments for editing transaction fee (EIP-1559 and legacy)
  • Add nav graph definitions

@gnosis/mobile-devs

@elgatovital elgatovital self-assigned this Jun 20, 2023
@elgatovital elgatovital marked this pull request as ready for review June 21, 2023 09:10
override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
super.onViewCreated(view, savedInstanceState)
with(binding) {

Copy link
Contributor

@biafra23 biafra23 Jun 21, 2023

Choose a reason for hiding this comment

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

Can this with(binding) be removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it's a placeholder for future pr. The whole fragment is empty right now.

Copy link
Contributor

@biafra23 biafra23 left a comment

Choose a reason for hiding this comment

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

Couldn't look at the screens, yet. But seems ok.

@elgatovital elgatovital merged commit 5618217 into master Jun 21, 2023
@elgatovital elgatovital deleted the feature/issue_1932 branch June 21, 2023 11:00
@github-actions github-actions bot locked and limited conversation to collaborators Jun 21, 2023
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.

2 participants