-
Notifications
You must be signed in to change notification settings - Fork 6
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
refactor: remove network from registerToken #WPB-9476 #3051
Conversation
7fee5b4
to
5296e63
Compare
Bencher Report
Click to view all benchmark results
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3051 +/- ##
===========================================
- Coverage 52.50% 52.50% -0.01%
===========================================
Files 1304 1304
Lines 50178 50182 +4
Branches 4679 4679
===========================================
+ Hits 26348 26349 +1
- Misses 21943 21947 +4
+ Partials 1887 1886 -1
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Datadog ReportBranch report: ✅ 0 Failed, 3106 Passed, 106 Skipped, 13.26s Total Time |
This reverts commit 44cd4fc.
Quality Gate passedIssues Measures |
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.
Thanks for taking care of some house cleaning 👏
* refactor: remove network from registerToken #WPB-9476 * cheeky commit singing test * Revert "cheeky commit singing test" This reverts commit 44cd4fc.
What's new in this PR?
Issues
removed Suppress and changed function so it doesn't expose network layer
Dependencies (Optional)
Same function is used here #3050
If this PR will be merged before those changes can be applied also there
PR Post Submission Checklist for internal contributors (Optional)
PR Post Merge Checklist for internal contributors
References
feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764
.