-
Notifications
You must be signed in to change notification settings - Fork 223
Add user display during short poll #6700
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
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
We detected some changes at Caution DO NOT create changesets for features which you do not wish to be included in the public changelog of the next CLI release. |
Coverage report
Show new covered files 🐣
Show files with reduced coverage 🔻
Test suite run success3544 tests passing in 1414 suites. Report generated by 🧪jest coverage report action from a17b645 |
43bda24 to
c79a291
Compare
fb8280d to
c59e0b5
Compare
c79a291 to
d5a3f88
Compare
c59e0b5 to
65302ac
Compare
packages/app/src/cli/services/bulk-operations/watch-bulk-operation.ts
Outdated
Show resolved
Hide resolved
jordanverasamy
left a comment
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.
looks good~
65302ac to
4b4ce44
Compare
d5a3f88 to
46c4d3b
Compare
4b4ce44 to
910f6b0
Compare
46c4d3b to
a17b645
Compare

Resolves: https://github.com/orgs/shop/projects/208/views/34?pane=issue&itemId=143907836&issue=shop%7Cissues-api-foundations%7C1160
WHY are these changes introduced?
Improve the user experience when polling for bulk operations by showing a task indicator.
WHAT is this pull request doing?
shortBulkOperationPollto display a task indicator while checking the bulk operation statusHow to test your changes?
ing progress