Skip to content

Commit bc75e47

Browse files
SarmisthaSarmistha
authored andcommitted
ACP2E-2709: [Feature Request] Customer suggests that Submit Comment Button on Order Details page is confusing and should be changed to something else
1 parent c203b8a commit bc75e47

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

app/code/Magento/Sales/Block/Adminhtml/Order/View/History.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ protected function _prepareLayout()
5959
$button = $this->getLayout()->createBlock(
6060
\Magento\Backend\Block\Widget\Button::class
6161
)->setData(
62-
['label' => __('Update Changes'), 'class' => 'action-save action-secondary', 'onclick' => $onclick]
62+
['label' => __('Update'), 'class' => 'action-save action-secondary', 'onclick' => $onclick]
6363
);
6464
$this->setChild('submit_button', $button);
6565
return parent::_prepareLayout();

app/code/Magento/Sales/i18n/en_US.csv

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -820,3 +820,4 @@ If set YES Email field will be required during Admin order creation for new Cust
820820
"Logo for PDF Print-outs","Logo for PDF Print-outs"
821821
"Please provide a comment text or update the order status to be able to submit a comment for this order.", "Please provide a comment text or update the order status to be able to submit a comment for this order."
822822
"A status change or comment text is required to submit a comment.", "A status change or comment text is required to submit a comment."
823+
"Update","Update"

dev/tests/integration/testsuite/Magento/Sales/Controller/Adminhtml/Order/ViewCommentTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public function testVerifyStatusCommentUpdateButtonLabel(): void
5252
$this->dispatch('backend/sales/order/view/');
5353
$content = $this->getResponse()->getBody();
5454
$this->assertStringContainsString(
55-
'<span>Update Changes</span>',
55+
'<span>Update</span>',
5656
$content
5757
);
5858
}

0 commit comments

Comments
 (0)