SF-3758 Fix crash when two users have the same Paratext id, sanitize import draft crashes#3768
SF-3758 Fix crash when two users have the same Paratext id, sanitize import draft crashes#3768RaymondLuong3 merged 2 commits intomasterfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3768 +/- ##
=======================================
Coverage 81.30% 81.30%
=======================================
Files 622 622
Lines 39382 39387 +5
Branches 6390 6414 +24
=======================================
+ Hits 32019 32025 +6
+ Misses 6378 6377 -1
Partials 985 985 ☔ View full report in Codecov by Sentry. |
RaymondLuong3
left a comment
There was a problem hiding this comment.
@RaymondLuong3 reviewed 4 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on pmachapman).
|
@pmachapman Can you update the title of the PR? I realize it's hard to title this well, but the actual changes are quite different from the title. |
@Nateowami Is this title better? |
5ae51c0 to
5d939f2
Compare
This PR fixes the root cause of the issue (a regression introduced by SF-2788 Reduce calls to the Paratext Registry (#2529)), and updated the SignalR notification to not include a stack trace.
This change is