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

Settings - Blockchain portfolio #217

Closed
wants to merge 20 commits into from
Closed

Conversation

a-novi
Copy link
Contributor

@a-novi a-novi commented Dec 6, 2019

No description provided.

src/redux/app/actions.ts Outdated Show resolved Hide resolved
src/redux/app/actions.ts Outdated Show resolved Hide resolved
src/redux/app/actions.ts Outdated Show resolved Hide resolved
src/redux/app/actions.ts Outdated Show resolved Hide resolved
src/redux/app/reducer.ts Outdated Show resolved Hide resolved
@a-novi a-novi force-pushed the settings-blockchain-portfolio branch from e3cefc4 to c3ddf94 Compare December 9, 2019 10:55
@a-novi a-novi requested a review from krisboit December 9, 2019 12:33
@a-novi a-novi force-pushed the settings-blockchain-portfolio branch 2 times, most recently from c3ddf94 to e3cefc4 Compare December 9, 2019 13:03
@codecov
Copy link

codecov bot commented Dec 9, 2019

Codecov Report

Merging #217 into master will decrease coverage by 0.25%.
The diff coverage is 21.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #217      +/-   ##
==========================================
- Coverage   46.33%   46.08%   -0.26%     
==========================================
  Files         170      170              
  Lines        2212     2198      -14     
  Branches      462      469       +7     
==========================================
- Hits         1025     1013      -12     
+ Misses       1182     1180       -2     
  Partials        5        5
Flag Coverage Δ
#unittests 46.08% <21.87%> (-0.26%) ⬇️
Impacted Files Coverage Δ
src/app.tsx 89.28% <ø> (-0.37%) ⬇️
src/components/splash-screen/SplashScreen.tsx 16.66% <ø> (ø)
src/screens/watch/watch.tsx 22.22% <ø> (+3.17%) ⬆️
src/navigation/navigation.tsx 86.95% <ø> (ø) ⬆️
src/components/splash-screen/styles.ts 0% <ø> (ø) ⬆️
src/components/header-left/styles.ts 100% <ø> (ø) ⬆️
src/components/header-right/styles.ts 100% <ø> (ø) ⬆️
src/screens/settings/settings.tsx 66.66% <0%> (-3.93%) ⬇️
...rc/screens/settings/blockchain-portfolio/styles.ts 0% <0%> (ø)
src/redux/app/reducer.ts 55.55% <0%> (-6.95%) ⬇️
... and 8 more

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 4ce1d14...d73c477. Read the comment docs.

Copy link
Contributor

@krisboit krisboit left a comment

Choose a reason for hiding this comment

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

something is wrong with the rebase, please review the changes and make sure you don't revert things from master

@@ -46,8 +46,6 @@ jobs:
- uses: actions/setup-ruby@v1
with:
ruby-version: "2.x"
- name: Generate firebase config file
Copy link
Contributor

Choose a reason for hiding this comment

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

this shouldn't be removed

@@ -88,8 +86,8 @@ jobs:
ssh-private-key: ${{ secrets.SSH_CERTS }}
- name: Add to host
run: ssh-keyscan gitlab.com >> ~/.ssh/known_hosts
- name: Generate firebase config file
Copy link
Contributor

Choose a reason for hiding this comment

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

same here

@@ -88,8 +86,8 @@ jobs:
ssh-private-key: ${{ secrets.SSH_CERTS }}
- name: Add to host
run: ssh-keyscan gitlab.com >> ~/.ssh/known_hosts
- name: Generate firebase config file
run: echo '${{ secrets.FB_CONFIG_IOS_BETA }}' > ios/GoogleService-Info.plist
- name: Checkout
Copy link
Contributor

Choose a reason for hiding this comment

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

this shouldn't be added back

@krisboit krisboit closed this Dec 10, 2019
@a-novi a-novi deleted the settings-blockchain-portfolio branch January 8, 2020 09:43
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.

3 participants