Skip to content

Commit 329ed0e

Browse files
committed
feature/doc — Little precision about why I mixed two API in order to process two related events.
1 parent 50c7141 commit 329ed0e

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

app/favourite/favourite.factory.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,8 @@
181181
// WARNING: when deleting, I'm using firebase directly because angularfire just give you the deleted key but not the actual
182182
// record. And you can't get it yourself because by the time you get notified, the local one is already deleted.
183183
// So in order to get best of both world and do what I want to do the easy way, I'm mixing here angularfire and pure firebase
184-
// code.
184+
// code which is kind of a good thing cause angularfire intent is not to replace the firebase sdk completely but just simplify
185+
// integration with angular when it makes sense.
185186
var ref = faves.$ref();
186187
ref.on('child_removed', function (snapshot) {
187188
var value = snapshot.val();

0 commit comments

Comments
 (0)