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

Indicate account registration #1928

Merged
merged 4 commits into from
Nov 27, 2019

Conversation

nambrot
Copy link
Contributor

@nambrot nambrot commented Nov 27, 2019

Description

Some commands require you to be a registered account first. Sometimes that it's easy to forget, so let's remind the user

Tested

  • Nope

@asaj asaj added the automerge Have PR merge automatically when checks pass label Nov 27, 2019
@codecov
Copy link

codecov bot commented Nov 27, 2019

Codecov Report

Merging #1928 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #1928    +/-   ##
=======================================
  Coverage    74.2%   74.2%            
=======================================
  Files         278     278            
  Lines        7673    7673            
  Branches      674     961   +287     
=======================================
  Hits         5694    5694            
  Misses       1862    1862            
  Partials      117     117
Flag Coverage Δ
#mobile 74.2% <ø> (ø) ⬆️

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 9a92271...3b1624c. Read the comment docs.

@celo-ci-bot-user celo-ci-bot-user merged commit ff6fb3e into master Nov 27, 2019
@celo-ci-bot-user celo-ci-bot-user deleted the nambrot/indicate-account-registration branch November 27, 2019 06:13
aaronmgdr added a commit that referenced this pull request Nov 27, 2019
* master:
  Fix celotool expected cluster check (#1932)
  Build Page for Baklava Launch Part 1  (#1866)
  cli: Fix for #1875 (#1920)
  cli: Fixes for #1880 and #1874 (#1918)
  Minor edits to full-node and other docs (#1917)
  Check that we have provisioned phone numbers (#1927)
  Sort invite contact list (#1936)
  Generate the pop for the attestation service with geth (#1931)
  Have validators use ephemeral node keys by default (#1922)
  Enhanced Validator Setup Docs (#1926)
  Indicate account registration (#1928)
  Bugfix: actually whitelist oracle account (#1865)
  Display current account locked gold requirement in lockedgold:show (#1923)
aaronmgdr added a commit that referenced this pull request Nov 27, 2019
* master: (97 commits)
  Fix celotool expected cluster check (#1932)
  Build Page for Baklava Launch Part 1  (#1866)
  cli: Fix for #1875 (#1920)
  cli: Fixes for #1880 and #1874 (#1918)
  Minor edits to full-node and other docs (#1917)
  Check that we have provisioned phone numbers (#1927)
  Sort invite contact list (#1936)
  Generate the pop for the attestation service with geth (#1931)
  Have validators use ephemeral node keys by default (#1922)
  Enhanced Validator Setup Docs (#1926)
  Indicate account registration (#1928)
  Bugfix: actually whitelist oracle account (#1865)
  Display current account locked gold requirement in lockedgold:show (#1923)
  Dedicated signing of attestation messages (#1913)
  Allow BLS info to be passed to validator:register without 0x prefix (#1919)
  Use cert manager instead of lego for k8s SSL certificates (#1924)
  Fix protocol tests (#1835)
  Fix bug in LockedGold.relock wrapper (#1915)
  Add account:show command (#1916)
  Add warning about misusing pumba (#1857)
  ...

# Conflicts:
#	packages/web/pages/_app.tsx
#	yarn.lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants