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

Sort invite contact list #1936

Merged

Conversation

Pedro-vk
Copy link
Contributor

Description

Sorting the list of contacts on invite section (inside settings) by name

Tested

Tested on Android phone

Related issues

@Pedro-vk Pedro-vk force-pushed the pedro-vk/#1514-invite-contacts-sorted branch from 8bfbac0 to 2b8a0e1 Compare November 27, 2019 14:33
@codecov
Copy link

codecov bot commented Nov 27, 2019

Codecov Report

Merging #1936 into master will decrease coverage by <.01%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1936      +/-   ##
=========================================
- Coverage    74.2%   74.2%   -0.01%     
=========================================
  Files         278     278              
  Lines        7673    7675       +2     
  Branches      961     674     -287     
=========================================
+ Hits         5694    5695       +1     
- Misses       1862    1863       +1     
  Partials      117     117
Flag Coverage Δ
#mobile 74.2% <66.66%> (-0.01%) ⬇️
Impacted Files Coverage Δ
packages/mobile/src/account/Invite.tsx 78.18% <66.66%> (-1.07%) ⬇️

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 21432c2...2b8a0e1. Read the comment docs.

@Pedro-vk Pedro-vk merged commit c57f23f into celo-org:master Nov 27, 2019
@Pedro-vk Pedro-vk deleted the pedro-vk/#1514-invite-contacts-sorted branch November 27, 2019 16:26
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Contacts when inviting are not sorted
2 participants