Skip to content

fix: added fixes for phantom docs, storage name resolution and subcollections#8

Open
lookieAU wants to merge 2 commits into
Flowdesktech:masterfrom
lookieAU:fix/subcollection-storage-phantom
Open

fix: added fixes for phantom docs, storage name resolution and subcollections#8
lookieAU wants to merge 2 commits into
Flowdesktech:masterfrom
lookieAU:fix/subcollection-storage-phantom

Conversation

@lookieAU

@lookieAU lookieAU commented Jul 20, 2026

Copy link
Copy Markdown

Description

Feature add-ons:

  1. Added nested documents/sub-collection feature -> expanding a document with a sub-collection now allows the user to expand as many nested sub-collections that are present

  2. Recursive delete -> Deleting a document now removes its sub-collections as well (added in confirmation pop-up as well)

Bugs:

  1. Fixed the phantom document bug -> Documents with no data but having sub-collections got dropped from the tree/list/json as get() was being used. However, admin SDK supports list documents which allow us to list phantom docs and their sub-collections (also works the same way on Firefoo)

  2. Fixed storage bug -> App hardcoded appspot.com to admin sdk based firebase storage URLs. Post-2024 these have changed to firebasestorage.app so it has been updated accordingly as well as fallback to the original suffix

Tests

  1. Ran full vittest suite on all new/changed files
  2. Manually tested fixes

Note

  1. package.json has a minor linting commit, safe to merge
  2. gitignore added bun.lock for devs using bun to commit instead of pnpm

@lookieAU lookieAU changed the title added fixes for phantom docs, storage name resolution and subcollections fix: added fixes for phantom docs, storage name resolution and subcollections Jul 20, 2026
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