Commit ef0df760 authored by Phil Hughes's avatar Phil Hughes

Merge branch 'aqualls-invalid-mr-message' into 'master'

Revise UI messages for invalid merge requests

See merge request gitlab-org/gitlab!75157
parents ce50e231 38e6b53e
- page_title "#{@merge_request.title} (#{@merge_request.to_reference}", _("Merge requests") - page_title "#{@merge_request.title} (#{@merge_request.to_reference}", _("Merge requests")
- badge_css_classes = "badge gl-text-white"
- badge_info_css_classes = "#{badge_css_classes} badge-info" - badge_start = '<span class="badge badge-pill gl-badge sm badge-info">'.html_safe
- badge_inverse_css_classes = "#{badge_css_classes} badge-inverse" - badge_end = '</span>'.html_safe
- err_fork_project_removed = s_("MergeRequest|Can't show this merge request because the fork project was deleted.")
- err_source_branch = s_("MergeRequest|Can't show this merge request because the source branch %{badge_start}%{source_branch}%{badge_end} is missing from project %{badge_start}%{project_path}%{badge_end}. Close this merge request or update the source branch.")
- err_target_branch = s_("MergeRequest|Can't show this merge request because the target branch %{badge_start}%{target_branch}%{badge_end} is missing from project %{badge_start}%{project_path}%{badge_end}. Close this merge request or update the target branch.")
- err_internal = s_("MergeRequest|Can't show this merge request because of an internal error. Contact your administrator.")
.merge-request .merge-request
= render "projects/merge_requests/mr_title" = render "projects/merge_requests/mr_title"
...@@ -11,20 +16,12 @@ ...@@ -11,20 +16,12 @@
.gl-alert-container .gl-alert-container
= sprite_icon('error', size: 16, css_class: 'gl-icon gl-alert-icon gl-alert-icon-no-title') = sprite_icon('error', size: 16, css_class: 'gl-icon gl-alert-icon gl-alert-icon-no-title')
.gl-alert-content{ role: 'alert' } .gl-alert-content{ role: 'alert' }
%p .gl-alert-body
We cannot render this merge request properly because
- if @merge_request.for_fork? && !@merge_request.source_project - if @merge_request.for_fork? && !@merge_request.source_project
fork project was removed = err_fork_project_removed
- elsif !@merge_request.source_branch_exists? - elsif !@merge_request.source_branch_exists?
%span{ class: badge_inverse_css_classes }= @merge_request.source_branch = err_source_branch.html_safe % { badge_start: badge_start, badge_end: badge_end, source_branch: @merge_request.source_branch, project_path: @merge_request.source_project_path }
does not exist in
%span{ class: badge_info_css_classes }= @merge_request.source_project_path
- elsif !@merge_request.target_branch_exists? - elsif !@merge_request.target_branch_exists?
%span{ class: badge_inverse_css_classes }= @merge_request.target_branch = err_target_branch.html_safe % { badge_start: badge_start, badge_end: badge_end, target_branch: @merge_request.target_branch, project_path: @merge_request.source_project_path }
does not exist in
%span{ class: badge_info_css_classes }= @merge_request.target_project_path
- else - else
of internal error = err_internal
%strong
Please close merge request or change branches with existing one
...@@ -21997,6 +21997,18 @@ msgstr "" ...@@ -21997,6 +21997,18 @@ msgstr ""
msgid "MergeRequest|Approved by @%{username}" msgid "MergeRequest|Approved by @%{username}"
msgstr "" msgstr ""
msgid "MergeRequest|Can't show this merge request because of an internal error. Contact your administrator."
msgstr ""
msgid "MergeRequest|Can't show this merge request because the fork project was deleted."
msgstr ""
msgid "MergeRequest|Can't show this merge request because the source branch %{badge_start}%{source_branch}%{badge_end} is missing from project %{badge_start}%{project_path}%{badge_end}. Close this merge request or update the source branch."
msgstr ""
msgid "MergeRequest|Can't show this merge request because the target branch %{badge_start}%{target_branch}%{badge_end} is missing from project %{badge_start}%{project_path}%{badge_end}. Close this merge request or update the target branch."
msgstr ""
msgid "MergeRequest|Compare %{target} and %{source}" msgid "MergeRequest|Compare %{target} and %{source}"
msgstr "" msgstr ""
......
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