-
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: handle the case where asset name can be missing [WPB-10830] 🍒 🍒 #3012
fix: handle the case where asset name can be missing [WPB-10830] 🍒 🍒 #3012
Conversation
…3002) * Commit with unresolved merge conflicts * merge conflicts --------- Co-authored-by: Mohamad Jaara <[email protected]>
…t-received-in-proteus-conversation-cherry-pick-cherry-pick
Quality Gate passedIssues Measures |
Test Results3 204 tests +4 3 099 ✔️ +4 3m 32s ⏱️ -17s Results for commit 3663189. ± Comparison against base commit 85e5c25. This pull request removes 1 and adds 5 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3012 +/- ##
===========================================
+ Coverage 52.49% 52.54% +0.05%
===========================================
Files 1301 1301
Lines 49977 50059 +82
Branches 4652 4666 +14
===========================================
+ Hits 26233 26304 +71
- Misses 21872 21879 +7
- Partials 1872 1876 +4
... and 2 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Datadog ReportBranch report: ✅ 0 Failed, 3099 Passed, 105 Skipped, 12.34s Total Time |
This PR was automatically cherry-picked based on the following PR:
Original PR description:
This PR was automatically cherry-picked based on the following PR:
Original PR description:
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
Causes (Optional)
in both cases if asset restriction is on then the message will not be handled correctly and in iOS case it will show prohibated asset, and in web case it will not be visable
Solutions
Needs releases with:
Testing
Test Coverage (Optional)
How to Test
Briefly describe how this change was tested and if applicable the exact steps taken to verify that it works as expected.
Notes (Optional)
Specify here any other facts that you think are important for this issue.
Attachments (Optional)
Attachments like images, videos, etc. (drag and drop in the text box)
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
.