-
Notifications
You must be signed in to change notification settings - Fork 41
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
15467 - Statement enhancements, remove duplicates - fix the sort order in pay as well. #1135
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1135 +/- ##
==========================================
+ Coverage 91.54% 91.57% +0.03%
==========================================
Files 153 163 +10
Lines 9842 11006 +1164
==========================================
+ Hits 9010 10079 +1069
- Misses 832 927 +95
Flags with carried forward coverage won't be shown. Click here to find out more.
|
85ead2f
to
f0f7b58
Compare
…t has some weird time difference.
…ld statements that no longer have settings
Kudos, SonarCloud Quality Gate passed!
|
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.
nice work 👍
Thank you Kial |
https://www.github.com/bcgov/issues/5467
Description of changes:
This gets rid of duplicate statements, without needing to have a query to remove duplicates.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the sbc-pay license (Apache 2.0).