The pull-request by BrianChapman (#130) mostly added support for onSnapshot, but had an issue where the onSnapshot callback sometimes wasn't called with the initial contents (if value was null at the time, or if called on a dec-ref instead of a query).
This pull-request is an updated version of his, fixing those issues.
EDIT: Apparently, BrianChapman made a newer pull-request (#145) with a fix for orderBy for it -- meaning we have two divergent pull-requests adding different fixes. If @soumak77 returns, one of us can integrate the other's fixes into a single onSnapshot merge. (too lazy atm since @soumak77 seems inactive currently)
The pull-request by BrianChapman (#130) mostly added support for onSnapshot, but had an issue where the onSnapshot callback sometimes wasn't called with the initial contents (if value was null at the time, or if called on a dec-ref instead of a query).
This pull-request is an updated version of his, fixing those issues.
EDIT: Apparently, BrianChapman made a newer pull-request (#145) with a fix for orderBy for it -- meaning we have two divergent pull-requests adding different fixes. If @soumak77 returns, one of us can integrate the other's fixes into a single onSnapshot merge. (too lazy atm since @soumak77 seems inactive currently)