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

Migrate from Algolia to Typesense Cloud #355

Merged
merged 8 commits into from
Oct 6, 2024
Merged

Migrate from Algolia to Typesense Cloud #355

merged 8 commits into from
Oct 6, 2024

Conversation

omarrida
Copy link
Contributor

@omarrida omarrida commented Oct 4, 2024 •
edited by techenby
Loading

Before the release:

  • Add the Typesense production env variables but keep the driver as Algolia until the release.
  • Merge (we disabled quick deploy temporarily).

During the release:

  • Enable maintenance mode.
  • Manually deploy via Forge.
  • Change driver to Typesense.
  • Run scout:import.
  • Confirm the number of records on Typesense = the number of records on Algolia.
  • Test it out (login with GitHub, add a package, confirm it's searchable, etc)
  • Lower maintenance mode
  • Have a great weekend!

Omar Rida and others added 6 commits August 13, 2024 21:54
@omarrida omarrida requested a review from techenby October 4, 2024 11:07
@omarrida omarrida marked this pull request as ready for review October 4, 2024 14:32
damiani
damiani reviewed Oct 4, 2024
View reviewed changes
Copy link

@damiani damiani left a comment

Choose a reason for hiding this comment

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

Looks good to me! Just one question about truncating readme/instructions, and a stray newline.

app/Package.php Outdated Show resolved Hide resolved
app/Http/Livewire/PackageList.php Outdated Show resolved Hide resolved
@omarrida
avoid truncating because algolias record limit doesnt apply anymore
7411140
@omarrida
- Remove usage of Algolia\SearchIndex in the novapackages API.
cb8dc13 
- Use the collection scout driver for testing.
- Enable the search API test class which was previously marked incomplete.
@techenby techenby merged commit a6a8a0f into main Oct 6, 2024
2 checks passed
@techenby techenby deleted the develop branch October 6, 2024 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damiani damiani damiani left review comments

@techenby techenby techenby approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@omarrida @damiani @techenby