Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove recently migrated transition pages from global search #4324

Closed
3 tasks done
dorothyyeager opened this issue Jan 13, 2021 · 2 comments · Fixed by #4420
Closed
3 tasks done

Remove recently migrated transition pages from global search #4324

dorothyyeager opened this issue Jan 13, 2021 · 2 comments · Fixed by #4420
Assignees
Milestone

Comments

@dorothyyeager
Copy link
Contributor

dorothyyeager commented Jan 13, 2021

We'll have some recently migrated pages on transition that are being removed and we don't want them to come up on the site search. Right now the index is using this to find which pages to index for transition: https://github.com/fecgov/fec-cms/blob/develop/fec/search/management/data/transition_pages.json.

Completion criteria

  • Take a look at transition site and make a list of pages we want to add to the global site search as well as pages that could be deleted. LIST IS HERE Content has looked at this list, and all transition pages on it. can be deleted.
  • Add/Delete items from transition_pages.json, and scrape. (Front-end)
  • We need to run a manual delete of items we want removed from the index using this documentation: http://gsa.github.io/slate/#delete-a-document (Front-end)

Completion criteria

The only thing being indexed from the transition site is the Audit Reports. (Everything else has been migrated)

@patphongs
Copy link
Member

This is now complete

@patphongs
Copy link
Member

Forgot that the PR needs to be merged before closing this issue.

@patphongs patphongs reopened this Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants