ctf0 / Laravel-Media-Manager

A "Vuejs & Laravel" Media Manager With Tons of Features
MIT License
829 stars 179 forks source link

Unknown Component in laravel 5.6 #46

Closed nyeyint closed 6 years ago

nyeyint commented 6 years ago

What version of Laravel are you using? (nothing lower than v5.4).

... Laravel 5.6

is it an issue related to installation or usage ?

...installation

what is the error you are getting ?

...unknown component

Extra Info "attaching a screen shot would be very helpful"

... untitled

ctf0 commented 6 years ago
nyeyint commented 6 years ago

Hi @ctf0 , Thanks for your reply. I already make all that steps.npm is also compile well but its show that errors.Do I need to register components in app.js file?

ctf0 commented 6 years ago

no, because resources/assets/vendor/MediaManager/js/manager.js already registers the comps globally.

can u dbl check the loaded js/css files in the browser ?

nyeyint commented 6 years ago

Thanks @ctf0.

I will check .Do you have any other demo project installation refrence?Its would be better for all others newbie developers who want to use your package. Thanks.

nyeyint commented 6 years ago

Hi @ctf0,

I just create repo for unknown component error. Demo Project with laravel 5.6

Everyting is already startup for your package.

nyeyint commented 6 years ago

Ok. I get it.all laravel fresh installation already included configuration in app.js.I just comment these two line


require('./bootstrap');

window.Vue = require('vue');


ctf0 commented 6 years ago

the only prob was having to init vue

  • make sure u have initiated ur vue ex.

new Vue({ 'el': '#app' })

i will add a note about that to the rdme, anyway ur problem is solved now ?

ctf0 commented 6 years ago

rdme updated https://github.com/ctf0/Laravel-Media-Manager/blob/master/README.md

nyeyint commented 6 years ago

@ctf0

Yeah my problem is solved now.Thanks.

nyeyint commented 6 years ago

I think your readme update is not perfact.I think error is bez of this line.

window.Vue = require('vue');

u can check in my demo repo.

ctf0 commented 6 years ago

i dont think so, anyway i've removed this line from the manager.js file so we now rely on the user app.js file

nyeyint commented 6 years ago

that great.Thanks for your discussion.