Commit be41aa28 authored by Amy Qualls's avatar Amy Qualls Committed by Enrique Alcántara

Improve UI message for unfinished pipelines

This message should be revised to meet our standards - include
current status, explanation, next step.

Update tests and translation file

Update the .pot file for translation, and update a test that checks
the output of the file we modified in this MR.

Update messages based on UX feedback

We've decided to NOT change one of the messages, and revising the
other. Update tests and translation files to match.

Update test and snapshot

The text was just a smidge off from what the tests were expecting.
Move the period inside the link, rather than outside of it, to strip
the extra space I'm seeing.

Update translations file

Update translations file to reflect changes to strings.

Changelog: changed
parent 42dd1dd2
...@@ -20,7 +20,7 @@ export default { ...@@ -20,7 +20,7 @@ export default {
}, },
i18n: { i18n: {
failedMessage: s__( failedMessage: s__(
`mrWidget|The pipeline for this merge request did not complete. Push a new commit to fix the failure, or check the %{linkStart}troubleshooting documentation%{linkEnd} to see other possible actions.`, `mrWidget|Merge blocked: pipeline must succeed. Push a commit that fixes the failure, or %{linkStart}learn about other solutions.%{linkEnd}`,
), ),
}, },
}; };
......
...@@ -241,7 +241,7 @@ This also applies if the pipeline has not been created yet, or if you are waitin ...@@ -241,7 +241,7 @@ This also applies if the pipeline has not been created yet, or if you are waitin
for an external CI service. If you don't use pipelines for your project, then you for an external CI service. If you don't use pipelines for your project, then you
should disable **Pipelines must succeed** so you can accept merge requests. should disable **Pipelines must succeed** so you can accept merge requests.
### "The pipeline for this merge request did not complete. Push a new commit to fix the failure or check the troubleshooting documentation to see other possible actions." message #### "Merge blocked: pipeline must succeed. Push a new commit that fixes the failure" message
This message is shown if the [merge request pipeline](pipelines/merge_request_pipelines.md), This message is shown if the [merge request pipeline](pipelines/merge_request_pipelines.md),
[merged results pipeline](pipelines/merged_results_pipelines.md), [merged results pipeline](pipelines/merged_results_pipelines.md),
......
...@@ -43483,6 +43483,9 @@ msgstr "" ...@@ -43483,6 +43483,9 @@ msgstr ""
msgid "mrWidget|Merge blocked: pipeline must succeed. It's waiting for a manual action to continue." msgid "mrWidget|Merge blocked: pipeline must succeed. It's waiting for a manual action to continue."
msgstr "" msgstr ""
msgid "mrWidget|Merge blocked: pipeline must succeed. Push a commit that fixes the failure, or %{linkStart}learn about other solutions.%{linkEnd}"
msgstr ""
msgid "mrWidget|Merge blocked: this merge request must be approved." msgid "mrWidget|Merge blocked: this merge request must be approved."
msgstr "" msgstr ""
...@@ -43600,9 +43603,6 @@ msgstr "" ...@@ -43600,9 +43603,6 @@ msgstr ""
msgid "mrWidget|The changes were not merged into" msgid "mrWidget|The changes were not merged into"
msgstr "" msgstr ""
msgid "mrWidget|The pipeline for this merge request did not complete. Push a new commit to fix the failure, or check the %{linkStart}troubleshooting documentation%{linkEnd} to see other possible actions."
msgstr ""
msgid "mrWidget|The source branch has been deleted" msgid "mrWidget|The source branch has been deleted"
msgstr "" msgstr ""
......
...@@ -57,7 +57,7 @@ RSpec.describe 'Merge request > User merges only if pipeline succeeds', :js do ...@@ -57,7 +57,7 @@ RSpec.describe 'Merge request > User merges only if pipeline succeeds', :js do
wait_for_requests wait_for_requests
expect(page).to have_css('button[disabled="disabled"]', text: 'Merge') expect(page).to have_css('button[disabled="disabled"]', text: 'Merge')
expect(page).to have_content('The pipeline for this merge request did not complete. Push a new commit to fix the failure, or check the troubleshooting documentation to see other possible actions.') expect(page).to have_content('Merge blocked: pipeline must succeed. Push a commit that fixes the failure, or learn about other solutions.')
end end
end end
...@@ -70,7 +70,7 @@ RSpec.describe 'Merge request > User merges only if pipeline succeeds', :js do ...@@ -70,7 +70,7 @@ RSpec.describe 'Merge request > User merges only if pipeline succeeds', :js do
wait_for_requests wait_for_requests
expect(page).not_to have_button 'Merge' expect(page).not_to have_button 'Merge'
expect(page).to have_content('The pipeline for this merge request did not complete. Push a new commit to fix the failure, or check the troubleshooting documentation to see other possible actions.') expect(page).to have_content('Merge blocked: pipeline must succeed. Push a commit that fixes the failure, or learn about other solutions.')
end end
end end
......
...@@ -16,7 +16,7 @@ exports[`PipelineFailed should render error message with a disabled merge button ...@@ -16,7 +16,7 @@ exports[`PipelineFailed should render error message with a disabled merge button
class="bold" class="bold"
> >
<gl-sprintf-stub <gl-sprintf-stub
message="The pipeline for this merge request did not complete. Push a new commit to fix the failure, or check the %{linkStart}troubleshooting documentation%{linkEnd} to see other possible actions." message="Merge blocked: pipeline must succeed. Push a commit that fixes the failure, or %{linkStart}learn about other solutions.%{linkEnd}"
/> />
</span> </span>
</div> </div>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment