Skip to content

Conversation

@tjsavage
Copy link
Contributor

@tjsavage tjsavage commented Jun 3, 2016

No description provided.

@tjsavage
Copy link
Contributor Author

tjsavage commented Jun 3, 2016

@ebidel WDYT? I'd like to get the Polymer Elements Catalog entries updated as soon as we can.

@cdata
Copy link
Contributor

cdata commented Jun 3, 2016

LGTM 👍

**Note: This element is for the older Firebase 2 API**
For the latest official Firebase 3.0-compatible component from the Firebase team,
see the [polymerfire](https://github.com/firebase/polymerfire) component.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You also might want to note that firebase-collection is replaced with firebase-query and link to https://github.com/firebase/polymerfire#firebase-query

@ebidel
Copy link
Contributor

ebidel commented Jun 3, 2016

LGTM. Want to add @chuckh 's comment?

@chuckh
Copy link
Contributor

chuckh commented Jun 4, 2016

LGTM 👍

@ebidel
Copy link
Contributor

ebidel commented Jun 4, 2016

LGTM Travis looks like it's having issues @cdata @tjsavage do you know what's up with it?

@ebidel ebidel merged commit 0e0815f into master Jun 4, 2016
@ebidel ebidel deleted the point-to-polymerfire branch June 4, 2016 17:41
@cdata
Copy link
Contributor

cdata commented Jun 5, 2016

@ebidel It's possible that the travis config for this repo is out of date. I'm not sure if Tedium runs on it or not.

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.

5 participants