-
Notifications
You must be signed in to change notification settings - Fork 44
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
Added notification if maytapi phone is not active #3077
Conversation
🚀 Deployed on https://deploy-preview-3077--glific-frontend.netlify.app |
…-frontend into enhancement/maytapi-status
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3077 +/- ##
==========================================
+ Coverage 86.01% 86.10% +0.08%
==========================================
Files 322 323 +1
Lines 10393 10402 +9
Branches 2216 2217 +1
==========================================
+ Hits 8940 8957 +17
+ Misses 1350 1342 -8
Partials 103 103 ☔ View full report in Codecov by Sentry. |
Glific Run #5221
Run Properties:
|
Project |
Glific
|
Branch Review |
enhancement/maytapi-status
|
Run status |
Passed #5221
|
Run duration | 24m 07s |
Commit |
871340e129 ℹ️: Merge 468df7bee6773c1d8d7528d2b7c5be207e1936b0 into 3588b8557ffc605f1ec43c114b3b...
|
Committer | Akansha Sakhre |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
169
|
View all changes introduced in this branch ↗︎ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good.
taget issue is glific/glific#3720
this PR is dependent on backend PR (
enhancement/maytapi_status
)