-
Notifications
You must be signed in to change notification settings - Fork 224
Release app bulk cancel command
#6811
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
base: main
Are you sure you want to change the base?
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
|
||
| if (response.bulkOperationCancel?.userErrors?.length) { | ||
| renderBulkOperationUserErrors(response.bulkOperationCancel.userErrors, 'Bulk operation cancellation errors.') | ||
| renderBulkOperationUserErrors(response.bulkOperationCancel.userErrors, 'Failed to cancel bulk operation.') |
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.
(Minor copy update, since I'm touching this anyway -- just wanted to make this a bit more descriptive!)
ericlee878
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.
Did simple 🎩. LGTM!
81b7d5f to
d9b8080
Compare
b429635 to
accddf0
Compare
|
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. |
accddf0 to
e1c7e25
Compare
Coverage report
Test suite run success3715 tests passing in 1438 suites. Report generated by 🧪jest coverage report action from 9d9884c |
da17e00 to
4e8c03f
Compare
d9b8080 to
534aadc
Compare
4e8c03f to
9d9884c
Compare

Follow-up to #6699. Don't merge until that's in and we've coordinated with CLI folks.
WHY are these changes introduced?
Address https://github.com/shop/issues-api-foundations/issues/1336.
WHAT is this pull request doing?
Unhide the
app bulk cancelcommand.How to test your changes?
Verify that the command is now visible! For example: