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

WIP: Making Twilio the default provider in the attestation service #2028

Merged

Conversation

aaitor
Copy link
Contributor

@aaitor aaitor commented Dec 4, 2019

Description

The documentation and script running the attestation service is not using Twilio as default provider. This PR changes that.

Tested

Running locally

Other changes

Describe any minor or "drive-by" changes here.

Related issues

  • Fixes #[issue number here]

Backwards compatibility

Brief explanation of why these changes are/are not backwards compatible.

@codecov
Copy link

codecov bot commented Dec 4, 2019

Codecov Report

Merging #2028 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2028      +/-   ##
==========================================
+ Coverage   74.42%   74.43%   +<.01%     
==========================================
  Files         281      281              
  Lines        7824     7822       -2     
  Branches      975      974       -1     
==========================================
- Hits         5823     5822       -1     
+ Misses       1884     1883       -1     
  Partials      117      117
Flag Coverage Δ
#mobile 74.43% <ø> (ø) ⬆️
Impacted Files Coverage Δ
packages/mobile/src/localCurrency/saga.ts 69.23% <0%> (+1.37%) ⬆️

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 272c4f0...bf57e69. Read the comment docs.

@aaitor aaitor added the automerge Have PR merge automatically when checks pass label Dec 6, 2019
@aaitor aaitor assigned nambrot and unassigned aaitor Dec 6, 2019
@celo-ci-bot-user celo-ci-bot-user merged commit d7c1368 into celo-org:master Dec 6, 2019
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.

5 participants