-
Notifications
You must be signed in to change notification settings - Fork 256
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
HCX Integration #1219
HCX Integration #1219
Conversation
Get ID from claim response identifier
Codecov ReportPatch coverage:
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## master #1219 +/- ##
==========================================
+ Coverage 56.12% 56.22% +0.09%
==========================================
Files 174 193 +19
Lines 8826 9489 +663
Branches 1525 1599 +74
==========================================
+ Hits 4954 5335 +381
- Misses 3818 4100 +282
Partials 54 54
... and 1 file with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Kudos, SonarCloud Quality Gate passed! |
@@ -28,6 +28,13 @@ def notification_task_generator(**kwargs): | |||
NotificationGenerator(**kwargs).generate() | |||
|
|||
|
|||
@celery.task() | |||
def send_webpush(**kwargs): |
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.
@vigneshhari Share your comments on this one.
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.
Changes Required
Co-authored-by: Mathew Alex <matthewzalex@gmail.com> Co-authored-by: Gigin George <mail.gigin@gmail.com>
No description provided.