Commit graph

631 commits

Author SHA1 Message Date
Satindar Dhillon
5d58cd28bf add local webreader for apple apps 2022-03-20 09:29:06 -07:00
Satindar Dhillon
af0d732ab6
Merge pull request #270 from omnivore-app/chore/ios-version-bump
Bump ios version to 1.4.1
2022-03-19 08:54:35 -07:00
Satindar Dhillon
e876f618ed bump ios version to 1.4.1 2022-03-19 08:06:45 -07:00
Satindar Dhillon
665ce1d754
Merge pull request #263 from omnivore-app/fix/remove-theme-duplicates
Remove extra theme objects in stitches config
2022-03-18 17:55:15 -07:00
Jackson Harper
efc9a30d1e
Merge pull request #264 from omnivore-app/fix/elastic-update-conflict
Fix/elastic update conflict
2022-03-18 16:33:45 -07:00
Hongbo Wu
705ebffff3 only refresh items on home feed 2022-03-18 17:08:47 +08:00
Hongbo Wu
4c535147f1 round reading progress to 100% if more than that
round reading progress to 100% if more than that

round reading progress to 100% if more than that
2022-03-18 14:56:49 +08:00
Hongbo Wu
17edc482f6 update debounce timeout to be 2 seconds 2022-03-18 14:06:58 +08:00
Hongbo Wu
365859bb63 import lodash 2022-03-18 12:30:06 +08:00
Hongbo Wu
1218bb0afd use lodash debounce the api call to save reading progress 2022-03-18 12:20:45 +08:00
Hongbo Wu
77050905a5 retry max three times if update labels is conflicted 2022-03-18 10:39:11 +08:00
Hongbo Wu
9d8d2e40d7 reduce time to save reading progress in elastic update 2022-03-18 10:29:01 +08:00
Hongbo Wu
ac567f7e71 retry max three times if update is conflicted 2022-03-18 10:17:20 +08:00
Hongbo Wu
7650a12be3
fix reading progress not updated after reading by updating cache (#258) 2022-03-18 09:58:08 +08:00
Jackson Harper
7910565dbf
Merge pull request #251 from omnivore-app/fix/jsonld-encoding
Use html decoding when getting values from fetched oembed
2022-03-17 16:53:21 -07:00
Satindar Dhillon
c8352576f2 remove extra theme objects in stitches config 2022-03-17 14:39:22 -07:00
Satindar Dhillon
dd70bc4a53
Merge pull request #262 from omnivore-app/fix/list-view-zstack-bug
Fix link card z axis layers
2022-03-17 12:28:02 -07:00
Satindar Dhillon
66c54569c8 fix link card z layers 2022-03-17 12:22:47 -07:00
Jackson Harper
0e1690d9f5
Merge pull request #246 from omnivore-app/fix/process-pool
Use a single process pool for all webviews
2022-03-17 11:44:43 -07:00
Jackson Harper
504e20f926
Merge pull request #259 from omnivore-app/fix/docker-compose-elastic
Add ELASTIC_URL env var to docker-compose
2022-03-17 11:32:57 -07:00
Jackson Harper
469c0ed764 Add ELASTIC_URL env var to docker-compose
This should fix startup of the API server in docker-compose.
2022-03-17 09:28:34 -07:00
Hongbo Wu
563131ea23
Fix/newsletter label not add (#256)
* default sort by savedAt to avoid recent read page to appear on top of the list

* revert lint on generated code

* fix elastic update script syntax error
2022-03-17 13:22:38 +08:00
Hongbo Wu
a2ce98229e
default sort by savedAt to avoid recent read page to appear on top of… (#254)
* default sort by savedAt to avoid recent read page to appear on top of the list

* revert lint on generated code
2022-03-17 10:54:00 +08:00
Hongbo Wu
af037a2837
make readingProgress required in the elastic page data (#253)
* make readingProgress required in the elastic page data

* delete readingProgress from function_resolvers because we have stored them in elastic
2022-03-17 10:06:21 +08:00
Jackson Harper
ff1200f3a1 Use html decoding when getting values from fetched oembed
If we fetch oembed data from an external source, instead of
handling it in readabilityjs we need to html decode it.
2022-03-16 15:29:42 -07:00
Jackson Harper
46f9306e17
Merge pull request #249 from omnivore-app/fix/elastic-reading-progress-nulls
Fix/elastic reading progress nulls
2022-03-16 14:35:45 -07:00
Jackson Harper
89f3719ba3 Fix linting 2022-03-16 14:23:39 -07:00
Jackson Harper
3d5a61285c
Merge pull request #247 from omnivore-app/fix/app-scrolling
In /app use the window to scroll to initial read position
2022-03-16 14:07:16 -07:00
Jackson Harper
0314d8cf17 Add some extra logging 2022-03-16 13:53:55 -07:00
Jackson Harper
dc27c0c7a4 Return 0 if looking up page reading progress fails 2022-03-16 13:33:36 -07:00
Jackson Harper
bef3dbd23a
Merge pull request #248 from omnivore-app/fix/elastic-reading-progress-nulls
Remove function handlers for reading progress
2022-03-16 13:18:29 -07:00
Jackson Harper
27143cbedf Remove function handlers for reading progress
These shouldn't be needed anymore because the values are stored
in elastic.
2022-03-16 13:11:36 -07:00
Jackson Harper
463e569e57 In /app use the window to scroll to initial read position
There isn't a scrollElementRef in `/app` because there is no
fixed header, so this will scroll the entire document instead.
2022-03-16 12:47:28 -07:00
Jackson Harper
88103cba9f
Merge pull request #162 from omnivore-app/dependabot/npm_and_yarn/apollo-datasource-3.3.1
Bump apollo-datasource from 0.7.3 to 3.3.1
2022-03-16 11:03:26 -07:00
Jackson Harper
aa2d107f97
Merge pull request #231 from omnivore-app/dependabot/npm_and_yarn/typeorm-0.2.45
Bump typeorm from 0.2.37 to 0.2.45
2022-03-16 10:44:04 -07:00
Jackson Harper
a58ccb7f14 Use a single process pool for all webviews
This is a first step to improving our caching on iOS.
2022-03-16 10:29:14 -07:00
Jackson Harper
a7c4659d90
Merge pull request #234 from omnivore-app/dependabot/npm_and_yarn/radix-ui/react-dialog-0.1.7
Bump @radix-ui/react-dialog from 0.1.5 to 0.1.7
2022-03-16 09:06:02 -07:00
Jackson Harper
3b9e1d36ae
Merge pull request #245 from omnivore-app/feature/add-label-to-newsletters
add Newsletter label to the page created by newsletters email
2022-03-16 08:59:24 -07:00
Hongbo Wu
07dcd5da26 add label to page only if not exists 2022-03-16 23:00:37 +08:00
Satindar Dhillon
39b19da07f
Merge pull request #243 from omnivore-app/chore/update-swift-five-six
Update to Swift 5.6
2022-03-16 07:53:37 -07:00
Hongbo Wu
1c4dcd7b00 automatically update updatedAt when page is updated in elastic 2022-03-16 22:28:39 +08:00
Hongbo Wu
f412758040 add Newsletter label to the page created by newsletters email 2022-03-16 18:40:05 +08:00
Hongbo Wu
27157006c1
use private bucket to upload page events (#244)
* use private bucket to upload page events

* fix tests

* add GCS_UPLOAD_PRIVATE_BUCKET in test env

* allow GCS_UPLOAD_PRIVATE_BUCKET to be empty
2022-03-16 14:39:07 +08:00
Jackson Harper
48e0a69931
Merge pull request #233 from omnivore-app/dependabot/npm_and_yarn/phosphor-react-1.4.1
Bump phosphor-react from 1.4.0 to 1.4.1
2022-03-15 22:13:44 -07:00
Satindar Dhillon
e25e6dc6c2 remove swiftformat warning 2022-03-15 21:52:17 -07:00
Satindar Dhillon
2a9ea0504e update to swift 5.6 2022-03-15 21:47:31 -07:00
Jackson Harper
e652a6ea8c
Rebased version of the elastic PR (#225)
* 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>
2022-03-16 12:08:59 +08:00
Jackson Harper
c4a3edc6be
Merge pull request #239 from omnivore-app/labels/db-migrations
Postgres migrations for the Elastic backend migration
2022-03-15 20:29:29 -07:00
Satindar Dhillon
d4a47259a4
Merge pull request #240 from omnivore-app/fix/ios14-refresh 2022-03-15 16:03:54 -07:00
Jackson Harper
2d258eaaf0 Remove unneeded file 2022-03-15 15:46:38 -07:00