Skip to content
This repository has been archived by the owner on Oct 19, 2018. It is now read-only.

unable to install on existing project #14

Open
34code opened this issue Nov 5, 2017 · 7 comments
Open

unable to install on existing project #14

34code opened this issue Nov 5, 2017 · 7 comments
Milestone

Comments

@34code
Copy link

34code commented Nov 5, 2017

gems/hyper-operation-0.5.12/lib/hyper-operation/call_by_class_name.rb:57:in `method_missing': undefined method `find_asset' for nil:NilClass (NoMethodError)

when trying: rails g hyperloop:install

I was trying to follow the webpacker tutorial on the website but failed..

@34code
Copy link
Author

34code commented Nov 5, 2017

I ran into a few issues since I'm not relying on active_record..

Seems like hyperloop has a strong an active_record dependency and I wasn't able to install individual gems like hyper-component either.

My fallback was to install the JS files and follow the JS import guides instead of the Rails guides..

@fzingg
Copy link
Member

fzingg commented Nov 6, 2017

For the error method_missing you got , it is a config error.
You should have set config.eager_load = false in config/development.rb or remove hyperloop dir from eager_load.

What do you mean by not relying on active_record and what are the issues you got.

@34code
Copy link
Author

34code commented Nov 13, 2017

I tried the tutorial after updating all my gems and it appears to be working now..
Tutorial link: http://ruby-hyperloop.org/tutorials/hyperlooprails/helloworld/

I get some warnings but loading the components seems to be working:

.gem/ruby/2.4.2/gems/hyper-react-0.12.7/lib/reactive-ruby/serializers.rb:1: warning: constant ::Bignum is deprecated
.gem/ruby/2.4.2/gems/hyper-react-0.12.7/lib/reactive-ruby/serializers.rb:1: warning: constant ::Fixnum is deprecated

@catmando
Copy link
Contributor

Okay lets explore having the hyperloop gem be a bit "smarter" such that if it does not detect active-record, it drops hyper-model, likewise for any other dependencies. I.e. it auto-adapts to whatever...

@catmando catmando added this to the Release 0.15 milestone Nov 15, 2017
@janbiedermann
Copy link
Contributor

i would first favour a config.require_hyper_model = true by default, likewise for other dependencies. later on this can then be made autodetect on app start.

@janbiedermann
Copy link
Contributor

janbiedermann commented Nov 20, 2017

also we could make the gems not depend on each other and let the gem ‘hyperloop’ then instead include the complete set.

@catmando
Copy link
Contributor

auto detect is no harder just do it...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants