-
Notifications
You must be signed in to change notification settings - Fork 94
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
[TNL-11886] Fix: Resolve Course Optimizer Scanning Bug #1666
base: master
Are you sure you want to change the base?
[TNL-11886] Fix: Resolve Course Optimizer Scanning Bug #1666
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1666 +/- ##
==========================================
+ Coverage 93.30% 93.33% +0.02%
==========================================
Files 1101 1107 +6
Lines 21856 22002 +146
Branches 4741 4753 +12
==========================================
+ Hits 20393 20535 +142
- Misses 1392 1396 +4
Partials 71 71 ☔ View full report in Codecov by Sentry. |
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! |
- Corrected LINK_CHECK_STATUSES.IN_PROGRESS value that is sent by backend.
b67fb2c
to
9a5f680
Compare
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.
I have tested and this is working fine. Good catch!
Make sure test-cases are working fine before merging |
TNL-11886
Corrected LINK_CHECK_STATUSES.IN_PROGRESS value that is sent by backend. As aligned with backend, it should be this.