KEYCLOAK-5391: Add node_modules back to repository for commons theme (k…

…eycloak#4457)

.gitignore

@@ -52,4 +52,6 @@ target
 
 # nodejs #
 ##########
-node_modules
+# KEYCLOAK-5391: We will re-exclude node_modules when node_modules handling is worked out.
+# For now, we keep our js libraries checked into GitHub, so we don't ignore.
+#node_modules

themes/pom.xml

@@ -53,6 +53,7 @@
 
     <build>
         <plugins>
+<!-- KEYCLOAK-5391: Commented out until node_modules fetching issue is resolved.
             <plugin>
                 <artifactId>maven-clean-plugin</artifactId>
                 <configuration>
@@ -80,7 +81,8 @@
                             <goal>yarn</goal>
                         </goals>
                         <configuration>
-                            <arguments>install --production=false --frozen-lockfile</arguments>
+                                               vvv remove space    vvv remove space on uncomment
+                            <arguments>install - -production=false - -frozen-lockfile</arguments>
                         </configuration>
                     </execution>
                 </executions>
@@ -91,6 +93,7 @@
                     <installDirectory>target</installDirectory>
                 </configuration>
             </plugin>
+-->
         </plugins>
     </build>
 

themes/src/main/resources/theme/keycloak/common/resources/README.md

@@ -6,7 +6,9 @@ libraries are not available in the public npm repo and are thus checked into
 GitHub.
 
 Javascript libraries under *./node_modules* directory are managed with yarn.  
-THEY SHOULD NOT BE CHECKED INTO GITHUB!
+
+THESE LIBRARIES SHOULD BE CHECKED INTO GITHUB UNTIL KEYCLOAK-5324 and KEYCLOAK-5392
+ARE RESOLVED.
 
 Adding or Removing javascript libraries
 ---------------------------------------