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

Fix/whatsapp handler can update urn on send #108

Merged
merged 3 commits into from
Apr 28, 2022

Conversation

rasoro
Copy link
Member

@rasoro rasoro commented Apr 28, 2022

referring to this PR nyaruka#437

@rasoro rasoro requested a review from matmsa27 April 28, 2022 11:24
@codecov-commenter
Copy link

Codecov Report

Merging #108 (e2ba3da) into main (96224b7) will decrease coverage by 0.01%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
- Coverage   70.95%   70.93%   -0.02%     
==========================================
  Files          96       96              
  Lines        8638     8647       +9     
==========================================
+ Hits         6129     6134       +5     
- Misses       1885     1889       +4     
  Partials      624      624              
Impacted Files Coverage Δ
handlers/test.go 0.00% <0.00%> (ø)
handlers/whatsapp/whatsapp.go 75.92% <100.00%> (+0.23%) ⬆️

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 96224b7...e2ba3da. Read the comment docs.

@rasoro rasoro merged commit 82b756f into main Apr 28, 2022
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.

3 participants