I encountered a number of issues while trying to get my own local dev
environment set up. This change adds missing information to the readme, updates
the Postgres image to one that includes required module dependencies, and
removes next.js rewrite rules (IS THIS SAFE?) which were no longer used and
causing issues.
* Add elastic to our docker compose
* add AND/OR/NOT search operations
* add elastic and create article in elastic
* change error code when elastic throws error
* add search pages in elastic
* add search by labels
* Add elastic to GitHub Action
* Update elastic version
* Fix port for elastic
* add url in search query
* Set elastic features when running tests
* add debug logs
* Use localhost instead of service hostname
* refresh elastic after create/update
* update search labels query
* add typescript support
* search pages in elastic
* fix search queries
* use elastic for saving page
* fix test failure
* update getArticle api to use elastic
* use generic get page function
* add elastic migration python script
* fix bulk helper param
* save elastic page id in article_saving_request instead of postgres article_id
* fix page archiving and deleting
* add tests for deleteArticle
* remove custom date type in elastic mappings which not exist in older version of elastic
* fix timestamp format issue
* add tests for save reading progress
* add tests for save file
* optimize search results
* add alias to index
* update migration script to receive env var as params
* Add failing test to validate we don't decrease reading progress
This test is failing with Elastic because we aren't fetching
the reading progress from elastic here, and are fetching it
from postgres.
* Rename readingProgress to readingProgressPercent
This is the name stored in elastic, so fixes issues pulling the
value out.
* Linting
* Add failing test for creating highlights w/elastic
This test fails because the highlight can't be looked up. Is there
a different ID we should be passing in to query for highlights,
or do we need to update the query to look for elastic_id?
* add tests code coverage threshold
* update nyc config
* include more files in test coverage
* change alias name
* update updateContent to update pages in elastic
* remove debug log
* fix createhighlight test
* search pages by alias in elastic
* update set labels and delete labels in elastic
* migration script enumeration
* make BULK_SIZE an env var
* fix pdf search indexing
* debug github action exit issue
* call pubsub when create/update/delete page in elastic
* fix json parsing bug and reduce reading data from file
* replace a depreciated pubsub api call
* debug github action exit issue
* debug github action exit issue
* add handler to upload elastic page data to GCS
* fix tests
* Use http_auth instead of basic_auth
* add index creation and existing postgres tables update in migration script
* fix a typo to connect to elastic
* rename readingProgress to readingProgressPercent
* migrate elastic_page_id in highlights and article_saving_request tables
* update migration script to include number of updated rows
* update db migration query
* read index mappings from file
* fix upload pages to gcs
* fix tests failure due to pageContext
* fix upload file id not exist error
* Handle savedAt & isArchived attributes w/out quering elastic
* Fix prettier issues
* fix content-type mismatching
* revert pageId to linkId because frontend was not deployed yet
* fix newsletters and attachment not saved in elastic
* put linkId in article for setting labels
* exclude orginalHtml in the result of searching to improve performace
* exclude content in the result of searching to improve performace
* remove score sorting
* do not refresh immediately to reduce searching and indexing time
* do not replace the backup data in gcs
* fix no article id defined in articleSavingRequest
* add logging of elastic api running time
* reduce home feed pagination size to 15
* reduce home feed pagination size to 10
* stop revalidating first page
* do not use a separate api to fetch reading progress
* Remove unused comment
* get reading progress if not exists
* replace ngram tokenizer with standard tokenizer
* fix tests
* remove .env.local
* add sort keyword in searching to sort by score
Co-authored-by: Hongbo Wu <hongbo@omnivore.app>
- Run on port 9090 so we don't conflict with other services
- Route the docker-compose requests to the host network
- Dont require preview bucket information on startup
This takes out some environment variables that are not needed
for new users testing out locally.
We also adjusted the port usage so docker-compose will use the
same ports that local dev typically uses.
