-
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
fix: secure message draft saving [WPB-11325] #3061
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3061 +/- ##
===========================================
+ Coverage 52.24% 52.51% +0.27%
===========================================
Files 1314 1314
Lines 50951 51013 +62
Branches 4724 4734 +10
===========================================
+ Hits 26617 26788 +171
+ Misses 22439 22327 -112
- Partials 1895 1898 +3
... and 5 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Datadog ReportBranch report: ✅ 0 Failed, 3132 Passed, 106 Skipped, 16.99s Total Time |
Bencher Report
Click to view all benchmark results
|
Quality Gate passedIssues Measures |
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
There were situations when saving message draft cause error in db
android.database.sqlite.SQLiteConstraintException: FOREIGN KEY constraint failed (code 787)
Causes (Optional)
Not working conversations when there was some saved draft
Solutions
Secure draft saving with checks needed to save a draft
Dependencies (Optional)
If there are some other pull requests related to this one (e.g. new releases of frameworks), specify them here.
Needs releases with:
Testing
Test Coverage (Optional)
How to Test