You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The Email Authorization documentation doesn't really make this clear, but this extension requires the DataTables JavaScript library as a dependency. Right now this dependency is defined as a Git submodule (see .gitmodules), so the command git submodule update --init needs to be called after the code is downloaded. Email Authorization is the only extension I know of in Canasta that has Git submodules (the others use Composer for their dependencies) - and, unfortunately, this command is not being called right now, so some of EA's special pages don't work.
I can think of various solutions to this problem:
Just add the hardcoded "git submodule" call to Dockerfile, to be called after extensions.yaml is handled.
Add some syntax to the extensions.yaml/skins.yaml format to indicate that an extension/skin has Git submodules, then add handling for that syntax (and add this syntax to the Email Authorization declaration).
Convince the EA developers (er, @cicalese) to add DataTables to composer.json as well, as a dependency, and then add EA to the composer.canasta.json file.
Locally patch EA to add DataTables to its composer.json, then add EA to composer.canasta.json.
Convince the EA developers to handle DataTables in what I would call the right way, which is to add this library directly to the extension code - the same way that other extensions (Cargo, Semantic Result Formats) include it. This prevents the code from breaking if there are any unexpected changes to DataTables in the future.
Of these, 1 is by far the easiest, though 5 is the best. @cicalese - any thoughts?
The text was updated successfully, but these errors were encountered:
Hopefully this works. It looks like a few other changes need to be made to EmailAuthorization before it can pass validation again, but anyway, this does seem to me like the best approach.
The Email Authorization documentation doesn't really make this clear, but this extension requires the DataTables JavaScript library as a dependency. Right now this dependency is defined as a Git submodule (see .gitmodules), so the command
git submodule update --init
needs to be called after the code is downloaded. Email Authorization is the only extension I know of in Canasta that has Git submodules (the others use Composer for their dependencies) - and, unfortunately, this command is not being called right now, so some of EA's special pages don't work.I can think of various solutions to this problem:
Of these, 1 is by far the easiest, though 5 is the best. @cicalese - any thoughts?
The text was updated successfully, but these errors were encountered: