-
Notifications
You must be signed in to change notification settings - Fork 2
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
bower dependency "sofa-q-service" doesn't work as expected #6
Comments
Hi @knitzie! What |
Hi @knitzie I can't add much to what Nick road except that we install all components via NPM so bower usage isn't battle tested. Apart from that it should work :) However, a word of caution regarding the Btw, pretty cool to see you building a custom app on top of sofa! |
As a follow up and totally off-topic, I noticed that you came up with angular wrappers for the bare sofa services here: https://github.com/knitzie/custom-sofa/blob/f3a85b271654f5b9e6b7b3454cfb5c564c98c0a7/app/scripts/factory.js Thumbs up on figuring out those parts on your own especially as we still lack lots of documentation. I just wanted to let you know that we mean to provide such angular wrappers ourselves but they are hidden in Example of the wrapped If you are interested in contributing such packages we would love to take your contributions. We have a https://github.com/sofa/sofa-component-seed Not trying to trick you into extra work but if you like to contribute to sofa that might be a good start :) |
First of all: Thanks for the answers and compliments ^^ @nickjanssen Here's my index file: https://github.com/knitzie/custom-sofa/blob/master/app/index.html @cburgdorf Good to know that it should work ;) And good to know about the current "in process" packages! |
Hi guys!
I'm currently building my customized app with the sofa SDK. When installing the sofa-checkout-service and all the dependencies with bower, at first everything is just fine. But in my index.html file an error occours. The sofa-q-service import got the wrong path:
At the moment I fix it manually and it works.
The text was updated successfully, but these errors were encountered: