Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Commits buttons: handle selected revision and support Perforce terminology#61408

Merged
peterguy merged 9 commits into
mainfrom
peterguy/add-revision-to-commits-link-repo-root
Mar 28, 2024
Merged

Commits buttons: handle selected revision and support Perforce terminology#61408
peterguy merged 9 commits into
mainfrom
peterguy/add-revision-to-commits-link-repo-root

Conversation

@peterguy

@peterguy peterguy commented Mar 27, 2024

Copy link
Copy Markdown
Contributor

Followup to PR #60909, which added a Commits button to the folders, and included the revision in it, exposing the shortcoming of the Commits button in the repo root page, which didn't include the revision.

Closes #60487

Test plan

When viewing a repository root page, click the Commits button and ensure that the URL includes the current revision (branch, tag or commit).

…page.

Followup to PR #60909, which added a Commits button to the folders,
and included the revision in it, expoing the shortcome of the Commits button
in the repo root page, which didn't include the revision.
@cla-bot cla-bot Bot added the cla-signed label Mar 27, 2024
@peterguy peterguy requested a review from a team March 27, 2024 00:06
@peterguy peterguy self-assigned this Mar 27, 2024
@peterguy peterguy added the team/code-search Issues owned by the code search team label Mar 27, 2024
@sourcegraph-bot

sourcegraph-bot commented Mar 27, 2024

Copy link
Copy Markdown
Contributor

📖 Storybook live preview

Comment thread client/web/src/repo/tree/TreePage.tsx Outdated
And add handling for Perforce depots, so that the
button shows "changelists" for Perforce instead
of "commits" and it navigates to `/-/changelists`
instead of to `/-/commits`.
@peterguy peterguy changed the title Add the current revision to the Commits button link on the repo root page Commits buttons: handle selected revision and support Perforce terminology Mar 27, 2024
Comment thread client/web/src/repo/tree/TreePage.tsx Outdated
Comment thread client/web/src/repo/utils.tsx Outdated
@peterguy peterguy merged commit 77918d4 into main Mar 28, 2024
@peterguy peterguy deleted the peterguy/add-revision-to-commits-link-repo-root branch March 28, 2024 22:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed team/code-search Issues owned by the code search team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Commits button on repo page should respect selected revision

4 participants