-
Notifications
You must be signed in to change notification settings - Fork 7
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
Support flow export notification #423
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #423 +/- ##
==========================================
- Coverage 79.22% 79.21% -0.01%
==========================================
Files 80 80
Lines 22016 22020 +4
Branches 750 750
==========================================
+ Hits 17443 17444 +1
- Misses 4562 4565 +3
Partials 11 11 ☔ View full report in Codecov by Sentry. |
68b7049
to
dd361d6
Compare
src/list/NotificationList.ts
Outdated
@@ -100,6 +100,9 @@ export class NotificationList extends TembaList { | |||
} else if (notification.export.type === 'ticket') { | |||
icon = Icon.tickets_export; | |||
body = 'Exported tickets'; | |||
} else if (notification.export.type === 'flows') { |
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.
how about definition
since they're not just flows?
contact
, message
, results
and definition
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.
Okay
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.
And the body can be Exported definitions
? @rowanseymour
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.
Sure
dd361d6
to
cd69bec
Compare
No description provided.