-
Notifications
You must be signed in to change notification settings - Fork 40
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
Fixed the input payload for updating organization flows #3049
Conversation
🚀 Deployed on https://deploy-preview-3049--glific-frontend.netlify.app |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3049 +/- ##
==========================================
- Coverage 80.29% 80.26% -0.04%
==========================================
Files 322 322
Lines 10364 10368 +4
Branches 2208 2210 +2
==========================================
Hits 8322 8322
- Misses 1401 1405 +4
Partials 641 641 ☔ View full report in Codecov by Sentry. |
Glific Run #5148
Run Properties:
|
Project |
Glific
|
Branch Review |
fix/org-flows
|
Run status |
Failed #5148
|
Run duration | 26m 26s |
Commit |
82f17104ef ℹ️: Merge cbb3e08b8c0ebe816f3c9f66b4f627b75ab19a3f into cab883c6bcce3d21d958d092a6cc...
|
Committer | Akansha Sakhre |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
1
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
168
|
View all changes introduced in this branch ↗︎ |
Tests for review
cypress/e2e/notification/notification.spec.ts • 1 failed test
Test | Artifacts | |
---|---|---|
Notification list > downloads csv report for contact import |
Test Replay
Screenshots
|
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.
fixes #3048