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.

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

Gardens via the REST API #339

Merged
merged 7 commits into from
Mar 14, 2024
Merged

Gardens via the REST API #339

merged 7 commits into from
Mar 14, 2024

Conversation

th0rgall
Copy link
Contributor

  • feat: ignore STP invoices & subscriptions
  • fix: handle temporary price IDs on invoices which lead to ignored WTMG events
  • fix: load gardens in chunks of 1000
  • fix: test rest api
  • build: add production devmode command
  • fix: add chunked REST garden loading + improve garden preloading

(previously uncommitted but deployed)
Attempt to fix the gardens not showing up on some devices, which doesn't have a clear cause, but may be due to an issue with the quantity of items requested from Firebase in one call (~2000-5000 now)

debug: show debugging information, more chunks, wait between chunks

test 100ms interval

test 1000ms

test 1500ms

1000 chunk size, 500ms

chunk size 250, no delay

attempt at not converting multiple times between data structures
Previously preloading was still dependent on isFetchingGarden being false
Copy link

github-actions bot commented Mar 14, 2024

Visit the preview URL for this PR (updated for commit 89de6c8):

https://wtmg-dev--pr339-fix-chunked-gardens-xhgdiwwg.web.app

(expires Thu, 21 Mar 2024 15:34:13 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7773005e481e8bc60fd60a58f2d818cc7378f95b

@th0rgall th0rgall merged commit 89a6cae into master Mar 14, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant