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.

Sign up for GitHub

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

Jump to bottom

feat: search v3 #1735

Draft
wants to merge 91 commits into
base: main
Choose a base branch
from
Draft

feat: search v3 #1735

wants to merge 91 commits into from

Conversation

abvthecity
Copy link
Contributor

@abvthecity abvthecity commented Oct 28, 2024 •
edited
Loading

This is a prototype / proposal of search v3 records (indexed to algolia) using trigger.dev.

Key objectives of this PR:

  1. define a new algolia record shape that enforces a common data structure across all record types. This will allow us to aggregate the results in a more predictable manner.
  2. bootstraps the visible_by parameter and writes tests around generating the different role combinations (for both record generation, and constructing search queries)
  3. implements a bulk indexing mechanism via trigger.dev, which will (1) load the docs from FDR, (2) convert to RootNode, (3) create algolia records, (4) bulk delete + create objects into algolia.

@vercel Vercel
Copy link

vercel bot commented Oct 28, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
rbac.ferndocs.com ❌ Failed (Inspect) Nov 13, 2024 10:17pm
1 Skipped Deployment
Name Status Preview Updated (UTC)
fern-shell ⬜️ Ignored (Inspect) Nov 13, 2024 10:17pm

@abvthecity abvthecity had a problem deploying to Preview - app.buildwithfern.com October 28, 2024 20:30 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app-dev.buildwithfern.com October 28, 2024 20:30 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app.buildwithfern.com October 28, 2024 20:49 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app-dev.buildwithfern.com October 28, 2024 20:49 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app-dev.buildwithfern.com October 28, 2024 20:50 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app.buildwithfern.com October 28, 2024 20:50 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app.buildwithfern.com October 28, 2024 20:55 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app-dev.buildwithfern.com October 28, 2024 20:55 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app-dev.buildwithfern.com October 29, 2024 00:21 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app.buildwithfern.com October 29, 2024 00:21 — with GitHub Actions Failure
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 29, 2024 •
edited
Loading

📦 Next.js Bundle Analysis for fern-platform-monorepo

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dsinghvi dsinghvi Awaiting requested review from dsinghvi dsinghvi will be requested when the pull request is marked ready for review dsinghvi is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@abvthecity