Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to Firebase API v3 #76

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

burn2delete
Copy link
Contributor

@burn2delete burn2delete commented Jul 6, 2016

Update to Firebase API v3

IN-PROGRESS

Started thread to track v3 progress.

Separation of concerns:

  • firebase-cljs - Contains cljs interop, is accepting PR's for clj interop as well (may get renamed in the future). CLJ -> JS conversion and accepts get-in style korks, does not attempt to coerce types into or out of Firebase DB.
  • matchbox - User friendly API, may handle coercion between types (via hydrate).

There are currently 4 variants to be supported Java, Node.js and client js and android.

Should eventually solve #74

@timothypratley
Copy link

I tried this out but couldn't quite get it working:

adzerk.boot_cljs.util.proxy$clojure.lang.ExceptionInfo$ff19274a: ERROR: No such namespace: cljsjs.firebase, could not locate cljsjs/firebase.cljs, cljsjs/firebase.cljc, or Closure namespace "cljsjs.firebase" in file /Users/timothypratley/.boot/cache/tmp/Users/timothypratley/git/voterx/1z61/e1mfji/js/app.out/firebase_cljs/promise.cljs at file /Users/timothypratley/.boot/cache/tmp/Users/timothypratley/git/voterx/1z61/e1mfji/js/app.out/firebase_cljs/promise.cljs

I don't really understand why, because as far as I can tell it should be working just fine... cljsjs.firebase is a dependency, it should just work.

@burn2delete
Copy link
Contributor Author

burn2delete commented Jul 11, 2016

Try just running the JVM version, there are missing namespaces there which might be causing issues for cljs.

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.

2 participants