-
Notifications
You must be signed in to change notification settings - Fork 286
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
Follow-up: Wait until survey trigger is complete #7222
Follow-up: Wait until survey trigger is complete #7222
Conversation
Size Change: -6 B (0%) Total Size: 1.38 MB
ℹ️ View Unchanged
|
Build files for 89b5d1d have been deleted. |
783862e
to
118ff33
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.
This looks good and works well in my testing. I'll push a few small tweaks but otherwise this is G2G! 👍
.dispatch( CORE_USER ) | ||
.triggerSurvey( triggerID ); | ||
const { response, error } = yield Data.commonActions.await( | ||
yield dispatch( CORE_USER ).triggerSurvey( triggerID ) |
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 yield is unneccessary – dispatch
returns a promise. yield
is for "raw" actions.
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.
Ah, my mistake for overlooking, thank you for correcting!
Ah, thank you so much for the corrections here, I've duly noted the mistakes and improvements for future reference, thank you @aaemnnosttv! |
Summary
Addresses issue:
Relevant technical choices
This follow-up PR aims to resolve this issue reported by the QA team, where User Input completion navigates the user to the Site Kit dashboard, but doesn't wait for the survey trigger to complete. Thus, the survey trigger request is often cancelled.
PR Author Checklist
Do not alter or remove anything below. The following sections will be managed by moderators only.
Code Reviewer Checklist
Merge Reviewer Checklist