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

Not Found Asset Dependency for AdminLTE #46

Closed
jjelev opened this issue Jun 26, 2015 · 26 comments
Closed

Not Found Asset Dependency for AdminLTE #46

jjelev opened this issue Jun 26, 2015 · 26 comments

Comments

@jjelev
Copy link

jjelev commented Jun 26, 2015

Hello all this issue relates to issue #6.
The exception which symfony returns is:
[InvalidArgumentException]
Unable to find file "@AvanzuAdminThemeBundle/Resources/public/vendor/adminlte/plugins/bootstrap-wysihtml5/bootstrap3-wysihtml5.js"

After researching on my own and nothing helped for me ( app/console avanzu:admin:fetch-vendor and even "bower update" in vendor/avanzu/admin-theme-bundle/Resources/bower/ directory) I found this.

I don't know how those dependencies are built but this array is not pointing to the correct file:

'admin_lte_wysiwyg'      => array(
    'inputs' => array(
        $lteJSBase.'plugins/bootstrap-wysihtml5/bootstrap3-wysihtml5.js',
    )
),

in vendor/avanzu/admin-theme-bundle/Resources/config/assets.php
The path is correct but filenames are different, should be
bootstrap3-wysihtml5.all.js

@valentinmagrez
Copy link

I m having the same issue, any update soon?

@jibundeyare
Copy link

@jjelev, @valentinmagrez in the meantime, check this PR #44

@OskarStark
Copy link
Contributor

have the same problem
bildschirmfoto 2015-07-21 um 10 29 24

can you please merge PR #44

@borisschapira
Copy link

Please, could you merge ?

@aftabnaveed
Copy link

It still seems to be not merged and pushed to packagist.org

@ogerik
Copy link

ogerik commented Aug 12, 2015

+1, please merge

@aftabnaveed
Copy link

+1 merge please

@OskarStark
Copy link
Contributor

@avanzu can you please merge this PR, otherwise the bundle cannot be used.

if there is feedback from your POV, let @jjelev know.

thank you

@quentin-st
Copy link
Collaborator

It would be great to add some maintainers to this project so we could merge PRs and push releases

@senol1
Copy link

senol1 commented Aug 28, 2015

I'm in agreement with you

@avanzu
Copy link
Owner

avanzu commented Aug 29, 2015

Any volunteers?
Atm. i just don't have the time to maintain or upgrade this bundle properly.

@quentin-st
Copy link
Collaborator

I volunteer

Joke aside, I wouldn't have time either to contribute as a main maintainer. But I could help answer issues/categorize them, merge some PRs too
Anyone would be ready to involve himself altogether?

@OskarStark
Copy link
Contributor

Trank you @chteuchteu

@OskarStark
Copy link
Contributor

Thank*

@quentin-st
Copy link
Collaborator

@OskarStark : would it be possible to publish a new release? We're not able to use AdminThemeBundle in its current state.
What about creating a GitHub organization with some maintainers, which would be able to merge PRs, answer & close issues? You would still have full control about what's happening on the project and make releases

What do you think about this?

@senol1
Copy link

senol1 commented Aug 31, 2015

I Think That it an excellent idea and I will be delighted to participated

2015-08-31 15:38 GMT+00:00 Quentin Stoeckel notifications@github.com:

@OskarStark https://github.com/OskarStark : would it be possible to
publish a new release? We're not able to use AdminThemeBundle in its
current state.
What about creating a GitHub organization with some maintainers, which
would be able to merge PRs, answer & close issues? You would still have
full control about what's happening on the project and make releases

What do you think about this?


Reply to this email directly or view it on GitHub
#46 (comment)
.

Sény Mbaye
Etudiant en Master II
Génie Informatique option Génie Logiciel / UFR Sciences et Technologies
Université de THIES
Portable : (+221)775915416
senyr9@gmail.com senyr9@gmail.com

@OskarStark
Copy link
Contributor

@chteuchteu i'm just a user as you, not a maintainer of this repo :)

@senol1
Copy link

senol1 commented Aug 31, 2015

ok I see .... anyway I managed to implement it but I download all of the
assets due myself

2015-08-31 17:57 GMT+00:00 Oskar Stark notifications@github.com:

@chteuchteu https://github.com/chteuchteu i'm just a user as you, not a
maintainer of this repo :)


Reply to this email directly or view it on GitHub
#46 (comment)
.

Sény Mbaye
Etudiant en Master II
Génie Informatique option Génie Logiciel / UFR Sciences et Technologies
Université de THIES
Portable : (+221)775915416
senyr9@gmail.com senyr9@gmail.com

@quentin-st
Copy link
Collaborator

@OskarStark Woops sorry, I must has been mislead by one of your previous messages...
@avanzu What do you think?

@avanzu
Copy link
Owner

avanzu commented Aug 31, 2015

New patch release is out with the recent fixes.
Tbh i'm really astonished by the constant interest for this bundle. Thanks for your support. :)
Anyways, forming an organization might be a bit over the top, collaborators seem to be more appropriate.

Hairsplitting aside, since @chteuchteu was kind enough to volunteer as tribute, welcome aboard 👍

@quentin-st
Copy link
Collaborator

Thank you @avanzu! I'll try to help regularly, I promise!

@OskarStark
Copy link
Contributor

Thank you

Not tried yet but is this Release available via packagist?

Oskar

@daften
Copy link

daften commented Sep 1, 2015

@OskarStark
Copy link
Contributor

great, thank you!

@OskarStark
Copy link
Contributor

@chteuchteu so can we close this issue then?

@quentin-st
Copy link
Collaborator

@OskarStark Yes indeed, it has been fixed by a11a2ee (tested yesterday)

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

No branches or pull requests