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

Remove Celo Default Recipient #9

Merged
merged 2 commits into from
Jul 18, 2019
Merged

Conversation

drklee3
Copy link
Contributor

@drklee3 drklee3 commented Jul 18, 2019

Description

Removes the Celo Default Recipient for the ph pilot build. Also added a noContacts message to show when both the contacts and search are empty to prevent No results for "" from showing.

Both of the following screenshots are after the Celo Default Recipient is removed.

Before After
Screenshot_1563473866 Screenshot_1563473696

Tested

Tests pass after updating the Invite snapshot.

Other changes

N/A

Related issues

@drklee3 drklee3 requested a review from annakaz July 18, 2019 22:03
@drklee3 drklee3 requested review from cmcewen and jmrossy as code owners July 18, 2019 22:03
@annakaz annakaz merged commit 337e1a4 into ph-pilot Jul 18, 2019
@drklee3 drklee3 deleted the drklee3/ph-default-recipient branch July 18, 2019 23:15
flashman1234 pushed a commit that referenced this pull request Jul 19, 2019
H34D referenced this pull request in H34D/celo-monorepo Nov 26, 2019
barbaraliau pushed a commit to barbaraliau/celo-monorepo that referenced this pull request Dec 11, 2020
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.

2 participants