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.

Sign up for GitHub

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

Jump to bottom

pref(web-color): add logo & title to landing page #29

Merged
merged 6 commits into from Jul 26, 2021
Merged

pref(web-color): add logo & title to landing page #29

merged 6 commits into from Jul 26, 2021

Conversation

jongbonga
Copy link
Contributor

@jongbonga jongbonga commented Jul 21, 2021 •
edited by PiDelport

@netlify
Copy link

netlify bot commented Jul 21, 2021 •
edited

✔️ Deploy Preview for nautilus-wallet ready!

🔨 Explore the source changes: 5832afe

🔍 Inspect the deploy log: https://app.netlify.com/sites/nautilus-wallet/deploys/60fe85822ec2390008d03dfc

😎 Browse the preview: https://deploy-preview-29--nautilus-wallet.netlify.app/

@jongbonga jongbonga self-assigned this Jul 21, 2021
@jongbonga jongbonga added the M: web-client Module: Web client app (Angular) label Jul 21, 2021
@codecov
Copy link

codecov bot commented Jul 21, 2021 •
edited

Codecov Report

Merging #29 (5832afe) into main (d139361) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##              main       #29   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines           27        27           
=========================================
  Hits            27        27

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 d139361...5832afe. Read the comment docs.

@PiDelport PiDelport added this to In progress in Nautilus Wallet via automation Jul 23, 2021
@PiDelport PiDelport moved this from In progress to Review in progress in Nautilus Wallet Jul 23, 2021
@PiDelport
Copy link
Collaborator

@PiDelport PiDelport marked this pull request as draft July 23, 2021 10:34
@jongbonga jongbonga marked this pull request as ready for review July 23, 2021 13:18
@jongbonga jongbonga marked this pull request as draft July 23, 2021 13:18
@jongbonga jongbonga marked this pull request as ready for review July 23, 2021 15:15
Nautilus Wallet automation moved this from Review in progress to Reviewer approved Jul 26, 2021
longtomjr
longtomjr approved these changes Jul 26, 2021
View reviewed changes
Copy link
Collaborator

@longtomjr longtomjr left a comment

Choose a reason for hiding this comment

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

Looks good. I assume the colors here match the designs.

@jongbonga
Copy link
Contributor Author

Looks good. I assume the colors here match the designs.

Yes

@jongbonga jongbonga merged commit b7ce99d into main Jul 26, 2021
Nautilus Wallet automation moved this from Reviewer approved to Done Jul 26, 2021
@jongbonga jongbonga deleted the landing branch July 26, 2021 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PiDelport PiDelport PiDelport approved these changes

@longtomjr longtomjr longtomjr approved these changes

@Nghondzweni Nghondzweni Awaiting requested review from Nghondzweni

Assignees

@jongbonga jongbonga

Labels
M: web-client Module: Web client app (Angular)
Projects
Nautilus Wallet
PRs: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jongbonga @PiDelport @longtomjr