Skip to content
This repository has been archived by the owner on Sep 10, 2022. It is now read-only.

Conflicting tag names for firebase v2/v3 elements #7

Open
jakemac53 opened this issue Jun 22, 2016 · 4 comments
Open

Conflicting tag names for firebase v2/v3 elements #7

jakemac53 opened this issue Jun 22, 2016 · 4 comments

Comments

@jakemac53
Copy link

Both the v2 and v3 firebase collections contain elements using the tags firebase-auth and firebase-document. This makes composing together multiple elements which use different firebase api versions impossible.

@ebidel
Copy link
Contributor

ebidel commented Jun 23, 2016

Use using multiple versions of the API recommended? That sounds like a bad
plan :)

On Wed, Jun 22, 2016, 10:05 AM Jacob MacDonald [email protected]
wrote:

Both the v2 and v3 firebase collections contain elements using the tags
firebase-auth and firebase-document. This makes composing together
multiple elements which use different firebase api versions impossible.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#7,
or mute the thread
https://github.com/notifications/unsubscribe/AAOigO30Np45zp1HOTV_eAZHRlOyq9KUks5qOWtWgaJpZM4I8AYQ
.

@jakemac53
Copy link
Author

This applies across the entire app, not just an individual component or package. It means that anybody upgrading from v2 has to migrate their whole app at once to v3, including all your dependencies. Elements are supposed to play nicely together and having conflicting tag names fundamentally goes against that.

In this case my guess is it doesn't matter in practice, because generally firebase is used at the top level of the app somewhere, and then the models from that are propagated down to other components. Thus, your dependencies don't typically use firebase directly so conflicts are a non-issue, or at least in small/medium sized projects.

Imo, it makes sense for these types of elements to put the major api version as part of the tag name (firebase-auth-v3 for instance).

@ebidel
Copy link
Contributor

ebidel commented Jun 23, 2016

This is precisely why the elements were released under a different organization. If users chose to use FB v3, they go to another place to get those elements.

@jakemac53
Copy link
Author

The issue I was pointing out though is that both elements are provided as part of this bundle. Maybe that is an argument against the bundle rules in general?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants