Skip to content
This repository has been archived by the owner on Jul 12, 2024. It is now read-only.

Remove custom search results #74

Merged
merged 4 commits into from
Nov 17, 2022
Merged

Remove custom search results #74

merged 4 commits into from
Nov 17, 2022

Conversation

Rosa-Fox
Copy link
Contributor

@Rosa-Fox Rosa-Fox commented Nov 17, 2022

Trello

Previously /brexit displayed search results that matched brexit-related terms. /corona displayed search results that matched coronavirus related terms. /search displayed live search results.

These pages are unloved and not used by anyone. Therefore they have been removed! The live search results still display as part of the / page dashboard.

How the /corona page looked before it was removed:

Screenshot 2022-11-17 at 12 20 16

/corona previously displayed live search results that matched
coronavirus related terms. The coronavirus dashboard isn't used anymore so
we have decided to remove it.
/brexit previously displayed live search results that matched
brexit related terms. The brexit dashboard isn't used anymore so
we have decided to remove it.
/search displayed live search results. The /search dashboard is no
longer used, so we have decided to remove it. The live search
results still display on the index page.
Copy link
Contributor

@gclssvglx gclssvglx left a comment

Choose a reason for hiding this comment

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

Nice - good work @Rosa-Fox - always good to see code being removed 👍

@Rosa-Fox Rosa-Fox merged commit 08c3572 into main Nov 17, 2022
@Rosa-Fox Rosa-Fox deleted the remove-custom-search-results branch November 17, 2022 14:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants