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

Clean seed words text area when returns from empty wallet view #1904

Merged
merged 3 commits into from
Dec 4, 2019

Conversation

Pedro-vk
Copy link
Contributor

Description

Clean seed words text area when returns from empty wallet view

Tested

Tested on Android after

Related issues

Backwards compatibility

Backward compatible

@Pedro-vk
Copy link
Contributor Author

BTW, I can't take screenshots because of security policies of the app

Copy link
Contributor

@jmrossy jmrossy left a comment

Choose a reason for hiding this comment

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

Can you use the imperative version of these APIs? they're more concise.
https://reactnavigation.org/docs/en/navigation-events.html

@jmrossy jmrossy assigned Pedro-vk and unassigned jmrossy Nov 27, 2019
@Pedro-vk Pedro-vk force-pushed the pedro-vk/clean-seed-words branch 3 times, most recently from edcc443 to b37f493 Compare November 29, 2019 11:31
@Pedro-vk Pedro-vk force-pushed the pedro-vk/clean-seed-words branch 11 times, most recently from f74ce75 to b54a644 Compare December 4, 2019 13:59
@codecov
Copy link

codecov bot commented Dec 4, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@c6e5b41). Click here to learn what that means.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1904   +/-   ##
=========================================
  Coverage          ?   74.43%           
=========================================
  Files             ?      281           
  Lines             ?     7822           
  Branches          ?      686           
=========================================
  Hits              ?     5822           
  Misses            ?     1883           
  Partials          ?      117
Flag Coverage Δ
#mobile 74.43% <50%> (?)
Impacted Files Coverage Δ
packages/mobile/src/import/ImportWallet.tsx 90.38% <40%> (ø)
packages/mobile/src/import/ImportWalletEmpty.tsx 79.41% <66.66%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c6e5b41...f2a14b9. Read the comment docs.

Copy link
Contributor

@jeanregisser jeanregisser left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Pedro-vk Pedro-vk merged commit d6d0086 into celo-org:master Dec 4, 2019
@Pedro-vk Pedro-vk deleted the pedro-vk/clean-seed-words branch December 4, 2019 14:29
aaronmgdr added a commit that referenced this pull request Dec 5, 2019
* master: (27 commits)
  Experience Brand Kit 1.0 (#1948)
  Adjust reference to the rewards app (#2065)
  [Wallet] Compatibility with exchange rate in string format (#2060)
  Fix Typo in CI config (#2056)
  Fix additional attestations instructions (#2057)
  Allow a specified address to disable/enable rewards distribution (#1828)
  Aaronmgdr/leaderboard patch (#2055)
  Move attestation service instructions to main page (#2051)
  Point To Updated Join Celo Video (#2052)
  Fix minor issue withe the ordering of instructions
  changes to docs related to discovery (#2025)
  [Docs] Fix typos in Running a Validator docs (#2045)
  Add node flag to celocli to set the target node for a single command (#2020)
  Fix broken links and spruce up CLI docs for accounts command (#2027)
  Prevent clipping of arrow component (#2036)
  Allocates an initial balance to the attestation bot (#2019)
  gold and dollar flags are required for faucet script (#1943)
  Clean seed words text area when returns from empty wallet view (#1904)
  Update validator script (#2026)
  Docs: PoS, metadata, gateway fee plus cleanup (#2022)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Wallet] Users SBAT see blank backup phrase screen when they choose to enter a new one
3 participants