diff options
| author | Earl Warren <earl-warren@noreply.codeberg.org> | 2024-07-29 17:32:33 +0200 |
|---|---|---|
| committer | Earl Warren <earl-warren@noreply.codeberg.org> | 2024-07-29 17:32:33 +0200 |
| commit | 4d2263e82e1b6a11213f43be67e49b8f15097d81 (patch) | |
| tree | 62e5bb4a00132c6d8a25be08632e0c6c8e99e7e7 | |
| parent | Merge pull request '[v8.0/forgejo] fix: never set to nil: poster of an issue ... (diff) | |
| parent | [UI] Show AGit label on merged PR (diff) | |
| download | forgejo-8.0.0.tar.xz forgejo-8.0.0.zip | |
Merge pull request '[v8.0/forgejo] [UI] Show AGit label on merged PR' (#4732) from bp-v8.0/forgejo-358ec80 into v8.0/forgejov8.0.0
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4732
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
| -rw-r--r-- | templates/repo/issue/view_title.tmpl | 9 | ||||
| -rw-r--r-- | tests/integration/git_test.go | 12 |
2 files changed, 21 insertions, 0 deletions
diff --git a/templates/repo/issue/view_title.tmpl b/templates/repo/issue/view_title.tmpl index 546c765d14..c6276279fe 100644 --- a/templates/repo/issue/view_title.tmpl +++ b/templates/repo/issue/view_title.tmpl @@ -69,6 +69,15 @@ <a {{if gt .Issue.PullRequest.Merger.ID 0}}href="{{.Issue.PullRequest.Merger.HomeLink}}"{{end}}>{{.Issue.PullRequest.Merger.GetDisplayName}}</a> <span class="pull-desc">{{ctx.Locale.TrN .NumCommits "repo.pulls.merged_title_desc_one" "repo.pulls.merged_title_desc_few" .NumCommits $headHref $baseHref $mergedStr}}</span> {{end}} + {{if .MadeUsingAGit}} + {{/* TODO: Move documentation link to the instructions at the bottom of the PR, show instructions when clicking label */}} + {{/* Note: #agit-label is necessary for testing whether the label appears when it should in tests/integration/git_test.go */}} + <a target="_blank" rel="noopener" href="https://forgejo.org/docs/latest/user/agit-support/"> + <span id="agit-label" data-tooltip-content="{{ctx.Locale.Tr "repo.pulls.agit_explanation"}}" class="ui small label"> + {{ctx.Locale.Tr "repo.pulls.made_using_agit"}} + </span> + </a> + {{end}} {{else}} {{if .Issue.OriginalAuthor}} <span id="pull-desc-display" class="pull-desc">{{.Issue.OriginalAuthor}} {{ctx.Locale.TrN .NumCommits "repo.pulls.title_desc_one" "repo.pulls.title_desc_few" .NumCommits $headHref $baseHref}}</span> diff --git a/tests/integration/git_test.go b/tests/integration/git_test.go index b7469a04cf..af6b825f0e 100644 --- a/tests/integration/git_test.go +++ b/tests/integration/git_test.go @@ -1067,6 +1067,18 @@ func doCreateAgitFlowPull(dstPath string, ctx *APITestContext, headBranch string }) t.Run("Merge", doAPIMergePullRequest(*ctx, ctx.Username, ctx.Reponame, pr1.Index)) + + t.Run("AGitLabelIsPresent Merged", func(t *testing.T) { + defer tests.PrintCurrentTest(t)() + + session := loginUser(t, ctx.Username) + + req := NewRequest(t, "GET", fmt.Sprintf("/%s/%s/pulls/%d", url.PathEscape(ctx.Username), url.PathEscape(ctx.Reponame), pr2.Index)) + resp := session.MakeRequest(t, req, http.StatusOK) + htmlDoc := NewHTMLParser(t, resp.Body) + htmlDoc.AssertElement(t, "#agit-label", true) + }) + t.Run("CheckoutMasterAgain", doGitCheckoutBranch(dstPath, "master")) } } |