Is the plan to checking npm modules like we do for other things? Or
actually pull npm modules from NPM? I want the approach here to be
consistent and not continue having two separate ways for new and old.
One problem with node_modules is that they include a lot of files that
should not be included in the distribution. We have a somewhat working
approach for the old stuff where we use a filter in Maven when copying
resources from node_modules directory created by Maven npm-update profile.
This is a requirement for new account console resources as well, as
otherwise we would significantly increase the size of the dist.
One issue with new account console is that it has some node_modules the old
stuff uses, but are different versions. Ideally we'd just have a single
package.json file for everything, but I don't think that's possible.
Ideally node_modules should all be in the common bundle and not directly in
the account theme, so they can be re-used by other things.
When we have a working build of the account console that addresses the
above (and anything else) it should be included in a default build (and
release) and shouldn't be a separate Maven profile to enable. We also need
to move it from experimental to preview.
On Tue, 29 Oct 2019 at 16:05, Bruno Oliveira <bruno(a)abstractj.org> wrote:
Good afternoon,
Alex is working on
https://issues.jboss.org/browse/KEYCLOAK-11293 and
I would like to double-check if we're ok with the requirements
described below.
What
The new account console needs to be built from Maven. The build also
needs to be able to build the RH-SSO version.
Why
There's no way to build the new account console from Maven and that's
a requirement that has an impact on testing and distribution.
How
- Build the new account console as part of the default distribution
(Keycloak and RHSSO build)
- The Maven build should be based on the steps described in the README
Acceptance Criteria
- Have all the artifacts from the new account console in the same
Keycloak/RHSSO server distribution
Additional information
This Jira can also incorporate
https://issues.jboss.org/browse/KEYCLOAK-5392 if possible, but that's
not mandatory
Comments are welcome, in the same Jira or here.
--
- abstractj
_______________________________________________
keycloak-dev mailing list
keycloak-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/keycloak-dev