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

search: deprecated quicklinks#40750

Merged
limitedmage merged 3 commits into
mainfrom
jp/removequicklinks
Aug 23, 2022
Merged

search: deprecated quicklinks#40750
limitedmage merged 3 commits into
mainfrom
jp/removequicklinks

Conversation

@limitedmage

Copy link
Copy Markdown
Contributor

Closes #39098

  • Quick links are no longer shown on the homepage if "Simple UI" is enabled (they were already not shown in the new right-side search sidebar)
  • The quicklink setting is now marked as deprecated
  • Quick link docs has been removed

Test plan

  • Test with Simple UI on and off to check that quicklinks show correctly in each setting

@cla-bot cla-bot Bot added the cla-signed label Aug 23, 2022
@limitedmage limitedmage requested a review from a team August 23, 2022 16:09
@sourcegraph-bot

sourcegraph-bot commented Aug 23, 2022

Copy link
Copy Markdown
Contributor

Codenotify: Notifying subscribers in CODENOTIFY files for diff f657a48...fed75f5.

Notify File(s)
@fkling client/web/src/search/home/SearchPageInput.tsx

@rvantonder rvantonder left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stampy stamp

@limitedmage limitedmage merged commit 9dd0bea into main Aug 23, 2022
@limitedmage limitedmage deleted the jp/removequicklinks branch August 23, 2022 22:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove quicklinks

3 participants