luyadev / luya

LUYA is a scalable web framework and content management system with the goal to please developers, clients and users alike.
https://luya.io
MIT License
811 stars 205 forks source link

Wrong namespace in story theme #2155

Closed hbugdoll closed 1 year ago

hbugdoll commented 3 years ago

There is a wrong namespace in asset file and view file of the recently added story theme for use with the luya-themes module.

Furthermore, it would be nice to have some clues in the readme about the different directory structure of this theme (obviously created by console command luya theme/create).

nadar commented 3 years ago

Would you like to send a PR regarding the wrong namespaces?

hbugdoll commented 3 years ago

I'd like to do it, but there is confusion about the different directory structure in regard to the others themes in the module. Instead of app\themes\story it should be something like luya\themes\themes\story, which doesn't work.

nadar commented 3 years ago

I think @boehsermoe is still working on that

hbugdoll commented 3 years ago

@nadar thanks for that information.

nadar commented 2 years ago

@boehsermoe is there something we can do here?

neoacevedo commented 2 years ago

The following error happens for all themes in the package:

Exception 'yii\base\InvalidConfigException' with message 'Theme base path have to be absolute or alias: vendor\luyadev/luya-themes\themes\escapeVelocity'

nadar commented 1 year ago

I am going to close this issue because:

If you think this is still important or there are more/new informations. Please reopen the issue and let us know.

If this is a problem or a feature you like to discuss, join our discussion: LUYA discussions. We try to keep our issue tracker as clean as possible in order to focus on active issues.