bug: a link from the search results lead to egghead.io about egghead-next HOT 5 CLOSED

anything /s is going to forward to the “api domain” because it is reserved for a url shortener

we’d originally used it in our naming convention for this site for search, but had to migrate to /q

https://share.getcloudapp.com/KoulNbZj

I recorded this to try and reproduce the issue and wasn’t able to. A screen recording would be useful.

from egghead-next.

@joelhooks - here's a recording: https://youtu.be/mGnB-L8AC28
One thing I didn't mention - I'm using brave as a browser. Might that be the issue?

from egghead-next.

Looks like there is something up with a couple items in Algolia as this particular video is on Page 1 of the results with the proper link and ~4 videos have the /s/* path.

I'm closing this, as it's not related to this application, but we'll look into it.

from egghead-next.

from egghead-next.

this felt like an indexing anomaly so I just scripted a little cleaner to remove the objects from Algolia:

import algoliasearch from 'algoliasearch'

const client = algoliasearch(APP_ID, API_SECRET)
const index = client.initIndex(INDEX_NAME)

const filter = {
  hitsPerPage: 1200,
  facetFilters: [
    'type:-lesson',
    'type:-course',
    'type:-podcast',
    'type:-talk',
    'type:-playlist',
  ],
}

const results = await index.search('', filter)

const offensiveIds = results.hits
  .filter((hit) => hit.type === null)
  .map((hit) => hit.objectID)

console.log(offensiveIds)

await index.deleteObjects(offensiveIds)

from egghead-next.