-
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 warning if flow is not published #3142
base: master
Are you sure you want to change the base?
Conversation
🚀 Deployed on https://deploy-preview-3142--glific-frontend.netlify.app |
Glific Run #5451
Run Properties:
|
Project |
Glific
|
Branch Review |
enhancement/flow-warning
|
Run status |
Passed #5451
|
Run duration | 24m 59s |
Commit |
62161d3451 ℹ️: Merge d7137353c9b66a31bceb16f57fa543d4665321db into 0b58bebe348db47c904e9944e1f5...
|
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 ↗︎ |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3142 +/- ##
==========================================
+ Coverage 81.73% 81.80% +0.06%
==========================================
Files 329 329
Lines 10564 10575 +11
Branches 2235 2239 +4
==========================================
+ Hits 8635 8651 +16
+ Misses 1316 1311 -5
Partials 613 613 ☔ View full report in Codecov by Sentry. |
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.
Tests are failing, can you please check? I have done another test run.
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.
I don't see any warning message.
Steps that I followed.
- created new flow and published
- tried editing the above flow and remove some node and clicked on menu links it navigated to the same without any warning.
May be I am missing something. Lets discuss during our call.
target issue is #3115