dappsnation / akita-ng-fire

Akita ❤️ Angular 🔥 Firebase
MIT License
131 stars 27 forks source link

syncWithRouter broken after latest fix? #194

Closed randallmeeker closed 3 years ago

randallmeeker commented 3 years ago

Is sync with router broken?

This change does not seem to match the type

https://github.com/dappsnation/akita-ng-fire/commit/dcf2281bcbee72cbab63cfd70fbbbc21fa349945

switchMap(params => this.syncCollection({ params: { params } })),

Additional context, here you can see a test setup with the following error

ERROR Error: Required parameter orgId from orgs/:orgId/projects doesn't exist in params {"params":{"orgId":"worldwide_productions"}}
fritzschoff commented 3 years ago

Thanks for letting us know. Will have a look tomorrow CET

fritzschoff commented 3 years ago

Thanks for all the reporting! Hope that should do the trick, will release a new minor version soon