-
Notifications
You must be signed in to change notification settings - Fork 2
Clear error state when successful request #277
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
Conversation
🦋 Changeset detectedLatest commit: fb041e0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
View your CI Pipeline Execution ↗ for commit fb041e0.
☁️ Nx Cloud last updated this comment at |
Deployed 0df3260 to https://ForgeRock.github.io/ping-javascript-sdk/pr-277/0df326011be4fc1537c878b861495656909040c7 branch gh-pages in ForgeRock/ping-javascript-sdk |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #277 +/- ##
==========================================
+ Coverage 50.26% 50.32% +0.05%
==========================================
Files 29 29
Lines 1711 1713 +2
Branches 195 194 -1
==========================================
+ Hits 860 862 +2
Misses 851 851
🚀 New features to boost your workflow:
|
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.
This looks good, but I think it needs to be rebased with main
as it contains commits that are likely already in the base branch.
72ca540
to
637e14b
Compare
637e14b
to
fb041e0
Compare
JIRA Ticket
https://pingidentity.atlassian.net/browse/SDKS-4042
Description
Error states should clear once we have successfully processed a success or next node.