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

Wordpress posts with a published gdocs successor are not excluded from search #2864

Closed
mlbrgl opened this issue Oct 27, 2023 · 1 comment
Closed

Comments

@mlbrgl
Copy link
Member

mlbrgl commented Oct 27, 2023

Expected behaviour

When a wordpress posts has been migrated over to gdocs and shares the same slug, it should be excluded from search.

Steps to reproduce

Make sure to not run actual indexing on Algolia then:

node --inspect itsJustJavascript/baker/algolia/indexToAlgolia.js

Inspect records

const records = await getPagesRecords()

There shouldn't be a "wordpress" entry for e.g. the "democracy" slug.

This is happening because of the leading /, which prevents slug matching from happening:

(post) => !publishedGdocsBySlug[`/${post.slug}`]

Screenshots

Screenshot 2023-10-27 at 09 58 59 Screenshot 2023-10-27 at 10 01 55 Screenshot 2023-10-27 at 10 02 56

Additional context

#2846

@ikesau
Copy link
Member

ikesau commented Nov 1, 2023

Fixed in #2846

@ikesau ikesau closed this as completed Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants