-
Notifications
You must be signed in to change notification settings - Fork 984
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
'Back up seed phrase' in Profile is present for recovered accounts #8938
Comments
Issue Status: 1. Open 2. Started 3. Submitted 4. Done Work has been started. These users each claimed they can complete the work by 2 years, 4 months ago. 1) acolytec3 has been approved to start work. Identify where seed-phrase suppression is failing in privacy_and_security_settings views.cljs and correct. Is there a preference for whether the whole backup seed phrase section is suppressed vs disabled? Learn more on the Gitcoin Issue Details page. |
@rachelhamlin Initial PR is submitted so can submit on Gitcoin whenever you have time to approve. :-) |
@rachelhamlin Can you (or @StatusSceptre) approve me on Gitcoin so I can submit work on this PR given that the PR was just merged? Thanks! |
Done @acolytec3! Will add your 20 DAI tip for the other issue where I shorted you on here. |
Issue Status: 1. Open 2. Started 3. Submitted 4. Done Work for 60.0 DAI (60.0 USD @ $1.0/DAI) has been submitted by: @StatusSceptre please take a look at the submitted work:
|
@rachelhamlin, any chance I can get paid for this plus the other 20 from #8027 side the PR is merged? |
Definitely & thanks for reminder @acolytec3. Gitcoin is currently not recognizing that Metamask is unlocked so I'm blocked. Going to try again later and escalate if necessary. |
@rachelhamlin Any luck here with metamask? |
Hi @acolytec3 unfortunately I'm still blocked on Metamask. :/ But another colleague is able to get in, so he'll pay you for this one + tip for #8027. Awesome work on #8960 too. I'll get help setting up a belated bounty there as well. |
@rachelhamlin, awesome, thanks! Hold off on #8951 till @yenda confirms my
approach is what he's thinking. #8960 is the one where the PR has been
merged already.
Any others that need attention right now?
…On Thu, Oct 17, 2019, 6:55 AM Rachel Hamlin ***@***.***> wrote:
Hi @acolytec3 <https://github.com/acolytec3> unfortunately I'm still
blocked on Metamask. :/ But another colleague is able to get in, so he'll
pay you for this one + tip for #8027
<#8027>.
Awesome work on #8951
<#8951> too. I'll get
help setting up a belated bounty there as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#8938>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEENFXF33PRHRLTFT66XGBDQPBADDANCNFSM4IUJH5NA>
.
|
⚡️ A tip worth 80.00000 DAI (80.0 USD @ $1.0/DAI) has been granted to @acolytec3 for this issue from @StatusSceptre. ⚡️ Nice work @acolytec3! Your tip has automatically been deposited in the ETH address we have on file.
|
Issue Status: 1. Open 2. Started 3. Submitted 4. Done This Bounty has been completed. Additional Tips for this Bounty:
|
Type: Bug
Summary:
Back up seed phrase
in Profile is present for recovered accounts. As well as after user backed up seed phrase еруBack up seed phrase
is still present and it lands user toYou're all set
screen.We need either to hide or disable
Back up seed phrase
option in Profile if it's not needed.Expected behavior
No
Back up seed phrase
in userProfile
for account which was recovered.No
Back up seed phrase
in userProfile
if it was already backed up.Actual behavior
Back up seed phrase
in userProfile
for account which was recovered is present.Back up seed phrase
in userProfile
is present if it was already backed up.Reproduction #1
Profile
->Privacy and Security
Reproduction #2
Profile
->Privacy and security
Additional Information
Develop 0.14.0 (2019090603)
The text was updated successfully, but these errors were encountered: