-
-
Notifications
You must be signed in to change notification settings - Fork 177
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
[#534] Demo project js makemakeAssertionOptions status changed to error #535
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #535 +/- ##
=======================================
Coverage 73.95% 73.95%
=======================================
Files 98 98
Lines 2638 2638
Branches 446 446
=======================================
Hits 1951 1951
Misses 586 586
Partials 101 101 ☔ View full report in Codecov by Sentry. |
@dotnet-policy-service agree |
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.
@yagiz-aydin I think this PR is missing some !== "ok" checks, could you replace all of them and I'll get this merged?
Closing this as it was superseeded by #551 |
On Demo project, We found when trying to open demo some js error occures.
We changed status code for
makemakeAssertionOptions.status
!== 'ok' to === 'error'Fully qualified demo, make sure we need to add https redirect on projects. It will also getting this error with the same way so, needs to maintain to HttpsRedirections
https://www.rfc-editor.org/rfc/rfc6454
Also related with this issue
#534