-
Notifications
You must be signed in to change notification settings - Fork 5
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 UG bugs found during PED #255
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #255 +/- ##
=========================================
Coverage 76.94% 76.94%
Complexity 515 515
=========================================
Files 75 75
Lines 1596 1596
Branches 143 143
=========================================
Hits 1228 1228
Misses 338 338
Partials 30 30 ☔ View full report in Codecov by Sentry. |
TODO: Write user-friendly explanation for SUBSTRING |
Solved by rephrasing explanation instead. |
TODO: replace remaining traces of "person" with "contact" |
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.
LGTM
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.
LGTM
…p into release-v1.4-ug
…-W12-3/tp into release-v1.4-ug
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.
LGTM
Features
section of UG #213+
to be entered anywhere. #200