-
Notifications
You must be signed in to change notification settings - Fork 47
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
feat: THES-105: Payment State Polling #771
Conversation
0a11351
to
f638579
Compare
Payment State polling on Submit Payment and Get Active Order, with a normally non-closeable Dialog.
f638579
to
d7b42ea
Compare
This spawned REV-3598.
… the Titan backend.
} catch (error) { | ||
// We dont quit on error. | ||
// yield call(handleErrors, error, true); | ||
yield put(paymentStateDataReceived({ state: PAYMENT_STATE.HTTP_ERROR })); |
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.
Add common logging for errors from New Relic.
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.
potentially HTTP errors should fall through, but nothing else.
src/payment/data/sagas.js
Outdated
// This shouldn't happen. | ||
// I don't think we need to banner... shouldn't our parent calls recover? (They invoke this) | ||
keepPolling = false; | ||
yield put(updatePaymentState.fulfill()); |
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.
throw exception?
Splitting exception handling in to another ticket. |
* feat: THES-105: Payment State Polling Payment State polling on Submit Payment and Get Active Order, with a normally non-closeable Dialog. * fix: Advisories permitted for the moment This spawned REV-3598. * fix: Removal of `PAYMENT_STATE.PROCESSING`, this state is internal to the Titan backend. * fix: Changes from PR Review
Payment State polling on Submit Payment and Get Active Order, with a normally non-closeable Dialog.
This spawned REV-3598.