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

Feature/security third party api mode #1609

Merged
merged 9 commits into from
Jul 2, 2020

Conversation

estebanmino
Copy link
Contributor

@estebanmino estebanmino commented Jun 1, 2020

Description

This PR adds a toggle under Security & Privacy settings to turn on / off the usage of third party APIs, as Etherscan to get incoming transactions in the user transaction history.

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

Issue

Resolves #???

@estebanmino estebanmino requested a review from a team as a code owner June 1, 2020 21:08
@estebanmino estebanmino changed the title [WIP] Feature/security third party api mode Feature/security third party api mode Jun 1, 2020
@estebanmino estebanmino added the needs-qa Any New Features that needs a full manual QA prior to being added to a release. label Jun 1, 2020
Copy link
Member

@andrepimenta andrepimenta left a comment

Choose a reason for hiding this comment

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

LGTM! 👍 👍

Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

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

For some reason it seems that the metametrics option is being disabled by default even though when I created my new wallet, I agreed and even further tried to enable it via settings but it doesn't stick; seen here = http://recordit.co/mKPBMVur9W

@ibrahimtaveras00 ibrahimtaveras00 added QA'd - Issues Found QA has been complete, however issues have been discovered that need to be addressed and removed needs-qa Any New Features that needs a full manual QA prior to being added to a release. labels Jul 2, 2020
@ibrahimtaveras00
Copy link
Contributor

That issue doesn't pertain to this PR as it happens on develop branch, so with that said this PR is QA Passed 👍

Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

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

👍

@ibrahimtaveras00 ibrahimtaveras00 added QA Passed A successful QA run through has been done and removed QA'd - Issues Found QA has been complete, however issues have been discovered that need to be addressed labels Jul 2, 2020
@estebanmino estebanmino merged commit 7706d93 into develop Jul 2, 2020
@estebanmino estebanmino deleted the feature/third-party-api-mode branch July 2, 2020 17:29
rickycodes pushed a commit that referenced this pull request Jan 31, 2022
* settings

* third party api mode

* strings

* spanish love

* tests

* test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next release QA Passed A successful QA run through has been done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants