Commit Graph

20 Commits

Author SHA1 Message Date
Abe Haskins
06a8ef9349 Upgraded to the Firebase 3.x.x SDK (#91) 2016-05-25 15:18:34 -07:00
jwngr
c251112d5b Fix bug with unbinding when component is unmounted 2015-09-29 10:56:34 -07:00
Rohit Kumar
14a2e7ae9e fix minor typo 2015-09-03 22:05:56 -07:00
jwngr
7c72395ebf Switched $key and $value to .key and .value 2015-07-08 19:07:12 -07:00
jwngr
d6d4a0d3a0 Fixed comment for _createRecord() 2015-07-08 17:10:22 -07:00
jwngr
f403f90188 Added $key to bound objects 2015-07-08 17:09:15 -07:00
jwngr
9124c34208 Fixed ordering issues with bindAsArray() and beefed up test suite 2015-07-07 00:22:55 -07:00
jwngr
dbbc5158e2 Switched test suite to Mocha and added linting 2015-07-05 20:38:35 -07:00
jwngr
f2fb34b17b Renamed $id to $key and fixed issue with sparse arrays 2015-07-03 16:05:00 -07:00
puf
2c014929e4 Fixed single quotes that travis dislikes 2015-06-02 11:42:10 -07:00
puf
4ef38406e6 Made existing test cases work, made arrays work 2015-06-02 11:24:32 -07:00
Frank van Puffelen
f098f7c133 dot notation 2015-04-26 16:35:55 -07:00
Frank van Puffelen
4fef3451de Use double quotes, according to the build server's wishes 2015-04-26 16:31:31 -07:00
Frank van Puffelen
fc7bb20591 Add key to the items in the array (as $id)
This way the key can be used to identify the items in the React output and tie updates back to the correct `child()`
2015-04-26 15:39:23 -07:00
Jonathan Buchanan
c3b88810f8 Allow a callback to be passed for the cancelCallback argument to firebaseRef.on()
Not sure how to add a test to your suite for this, but have checked it at least works locally in react-hn against the Hacker News API, which errors when attempting to load delayed comments
2014-10-23 13:57:05 +01:00
jacobawenger
36b01f9cf5 Added error codes to all errors and updated test suite 2014-09-17 11:54:02 -07:00
jacobawenger
25f9b59b03 Improved test coverage
- Got rid of JSXTransformer in test suite since it is not needed
- Simplified firebaseRef validation (we ran into a Node bug with this
old validation in GeoFire so we should change it here as well)
- Added a bunch of tests to improve code coverage
- Simplified existing tests
2014-09-13 18:48:10 -07:00
jacobawenger
e6f6d877bd Renamed source and spec files 2014-09-03 22:40:37 -07:00
danielmahal
a3ced1f25c Allow multiple listeners for identical paths
The current version of ReactFire doesn't cope well with listening to the same path in multiple components. This is because the listener reference from `ref.on('value')` is never stored, and thus all listeners are removed when `ref.off('value')` is called without a listener parameter.

This patch stores each `ref.on('value')` listener and passes it to `ref.off('value', listener)`.
2014-09-03 11:17:25 +02:00
jacobawenger
9d51a216f3 Fixed issue with limit() queries and added test suite
- Fixed issue when binding to Firebase limit() query
- Added error checking
- Added basic test suite
- Added travis integration
- Updated README
- Added limit query to todoApp demo
2014-06-29 14:17:39 -07:00