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

Switch to Firestore Lite #523

Merged
merged 37 commits into from
Feb 15, 2024
Merged

Switch to Firestore Lite #523

merged 37 commits into from
Feb 15, 2024

Conversation

ghinda
Copy link
Member

@ghinda ghinda commented Feb 15, 2024

Changes:

  • Switch to Firestore Lite, and upgrade to latest firebase version.
  • Manual Firestore sync using regular queries instead of snapshots.
  • Add a button to trigger immediate sync from the browser action popup.
  • Implement dashboard events to trigger automatic collection sync.
  • Disable persistence for the background script.
  • Add TM to Gmail strings in extension title and description.

ghinda added 30 commits February 8, 2024 10:44
@ghinda
Upgrade outdated.
62e9485
@ghinda
Upgrade css-minimizer-webpack-plugin.
b9779ad
@ghinda
Use initializeAuth instead of getAuth for firebase auth, to tree-shak…
313516d 
…e most of it away.
@ghinda
Set popupRedirectResolver: undefined, since we don't use it in auth.
28038b3
@ghinda
Bump version.
b901f5f
@ghinda
Tweak css injection methods for manifest v3.
f994be2
@ghinda
Support browser action methods in manifest v3.
1316ced
@ghinda
Use manifest v2 by default.
656a124
@ghinda
Upgrade mocha.
5b502ce
@ghinda
Initial changes for not relying on firestore snapshots anymore.
9401d20
@ghinda
Always load data from the background when opening the dialog.
6613c24
@ghinda
Clean up older snapshot related functionality.
8646daa
@ghinda
Switch to firestore/lite.
2f37b8b
@ghinda
Disable persistance for the background script on manifest v2.
32bf10d
@ghinda
Initial work on dashboard events.
57eb847
@ghinda
Fix issues with not clearning up the template cache.
44c4f81
@ghinda
Upgrade chai.
b0b151c
@ghinda
Add TM to the Gmail string in the extension name and description.
ac39586
@ghinda
Fix dashboard events client not working on briskine.com.
c96d567
@ghinda
Add share_all to list of default settings.
7ac4796
@ghinda
Remove the auto clear collection cache when opening the popup.
e10ba49
@ghinda
Add a manual sync button in the popup dashboard, to refetch all colle…
10e983b 
…ctions.

Also completely refetch collections when getting postmessage events from the dashboard.
@ghinda
Increase batched updates timer for dashboard events.
7b508bc
@ghinda
Styling for the new manual sync functionality.
4f4c2d6
@ghinda
Don't refetch templates for free users.
37c3bc9
@ghinda
Auto sync is last sync was more than 1 hour ago.
4a90d89
@ghinda
Tweak uppercase title on account settings link.
60dfd4f
@ghinda
Improve text overflow ellipsis on the popup footer email address.
ad93bd7
@ghinda
Fix loading indicator circle centering.
86d79e3
@ghinda
Revert to the old version of keyboard that uses content-script-cached…
d7f8642 
… templates.

Speeds up keyboard template insert.
@ghinda
Revert to the old version of dialog, that caches templates locally.
feeedaa 
Speeds up rendering when opening the dialog.
Templates will be refreshed anyway on templates-updated.
@ghinda
Upgrade firebase-tools.
8b742a0
@ghinda
Upgrade to latest firebase.
3cc4abc 
Hanging query should no longer be an issue since we don't use rpc anymore, after switching to firestore lite.
@ghinda
Add a newline to the last sync tooltip.
de76e7d
@ghinda
Attach the dashboard events message handler only on the dashboard.
45449f6
@ghinda
Avoid multiple parallel refetches.
8a3c4a5
@ghinda
Improve current url detection for blacklist, to work for nested ifram…
8deb833 
…es with no/different urls.

Only respond to the status event in the dashboard.
@ghinda ghinda self-assigned this Feb 15, 2024
@ghinda ghinda merged commit 0020349 into master Feb 15, 2024
1 check passed
@ghinda ghinda deleted the ghinda--firestore-lite branch February 15, 2024 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ghinda ghinda

Labels
None yet
Milestone
No milestone
Development

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