Re: [keycloak-dev] [KEYCLOAK-11293] Create build for new account console

On 10/29/2019 11:12 AM, Stian Thorgersen wrote: > 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. It's my understanding that the plan is to do it the same way we do for the admin console (check 'em in). When something better comes along (probably from PNC), then we will switch everything to use the new tools. > 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. Yes, that needs to be part of the requirements. > 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. I don't think we have time to address that before tech preview. But we should definitely take a hard look before final release. > 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. I'm not sure it even needs to be a separate profile any longer. I can't think of any reason to keep it in a separate profile once the npm libraries are checked in to github. > > On Tue, 29 Oct 2019 at 16:05, Bruno Oliveira <bruno(a)abstractj.org&gt; 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 >> > _______________________________________________ > keycloak-dev mailing list > keycloak-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/keycloak-dev _______________________________________________ keycloak-dev mailing list keycloak-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/keycloak-dev