tiagobento commented on code in PR #2532:
URL: 
https://github.com/apache/incubator-kie-tools/pull/2532#discussion_r1723916717


##########
packages/sonataflow-deployment-webapp/README.md:
##########
@@ -21,9 +21,9 @@ This is the web application for SonataFlow Deployments.
 
 ## Run local web server
 
-To run the local web server, use the following command:
+To run the local web server and the mock server, use the following command:
 
-    pnpm start
+    pnpm dev

Review Comment:
   I know this is not written anywhere, but we're defaulting to `start` for 
development purposes, as this is kind of a standard in the JS ecosystem...



##########
packages/sonataflow-management-console-webapp/README.md:
##########
@@ -0,0 +1,31 @@
+# sonataflow-management-console-webapp
+

Review Comment:
   Please describe this package in a few words, targeting users who may consume 
it.



##########
packages/sonataflow-management-console-image-env/README.md:
##########
@@ -0,0 +1,48 @@
+<!--
+   Licensed to the Apache Software Foundation (ASF) under one
+   or more contributor license agreements.  See the NOTICE file
+   distributed with this work for additional information
+   regarding copyright ownership.  The ASF licenses this file
+   to you under the Apache License, Version 2.0 (the
+   "License"); you may not use this file except in compliance
+   with the License.  You may obtain a copy of the License at
+     http://www.apache.org/licenses/LICENSE-2.0
+   Unless required by applicable law or agreed to in writing,
+   software distributed under the License is distributed on an
+   "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+   KIND, either express or implied.  See the License for the
+   specific language governing permissions and limitations
+   under the License.
+-->
+
+## @kie-tools/sonataflow-management-console-image-env
+
+---
+
+Apache KIE (incubating) is an effort undergoing incubation at The Apache 
Software
+Foundation (ASF), sponsored by the name of Apache Incubator. Incubation is
+required of all newly accepted projects until a further review indicates that
+the infrastructure, communications, and decision making process have stabilized
+in a manner consistent with other successful ASF projects. While incubation
+status is not necessarily a reflection of the completeness or stability of the
+code, it does indicate that the project has yet to be fully endorsed by the 
ASF.
+
+Some of the incubating project’s releases may not be fully compliant with ASF
+policy. For example, releases may have incomplete or un-reviewed licensing
+conditions. What follows is a list of known issues the project is currently
+aware of (note that this list, by definition, is likely to be incomplete):
+
+- Hibernate, an LGPL project, is being used. Hibernate is in the process of 
relicensing to ASL v2
+- Some files, particularly test files, and those not supporting comments, may 
be missing the ASF Licensing Header
+-
+
+- Hibernate, an LGPL project, is being used. Hibernate is in the process of
+  relicensing to ASL v2
+- Some files, particularly test files, and those not supporting comments, may
+  be missing the ASF Licensing Header

Review Comment:
   Duplicated content... please see 
https://github.com/apache/incubator-kie-tools/commit/be1e6e32bd32b50b2a740f3db189eda0b94609d0
 and verify other places here to not re-introduce this problem.



##########
packages/kogito-management-console/README.md:
##########
@@ -70,18 +70,18 @@ This package contains the `Containerfile/Dockerfile` and 
scripts to build a cont
 
    [comment]: <> (//TODO: Use EnvJson.schema.json to generate this 
documentation somehow.. See https://github.com/kiegroup/kie-issues/issues/16)
 
-   |                          Name                          |                  
        Description                          |                                  
         Default                                            |
-   | :----------------------------------------------------: | 
:-----------------------------------------------------------: | 
:------------------------------------------------------------------------------------------:
 |
-   |   `RUNTIME_TOOLS_MANAGEMENT_CONSOLE_KOGITO_ENV_MODE`   | Env Mode: "PROD" 
or "DEV". PROD enables Keycloak integration. |                                  
          "PROD"                                            |
-   |   `RUNTIME_TOOLS_MANAGEMENT_CONSOLE_KOGITO_APP_NAME`   |                 
Management Console app name.                  | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
-   | `RUNTIME_TOOLS_MANAGEMENT_CONSOLE_KOGITO_APP_VERSION`  |                
Management Console app version.                | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
-   | `RUNTIME_TOOLS_MANAGEMENT_CONSOLE_DATA_INDEX_ENDPOINT` |        The URL 
that points to the Data Index service.         | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
-   |    `KOGITO_CONSOLES_KEYCLOAK_DISABLE_HEALTH_CHECK`     |                
Disables Keycloak health-check.                | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
-   |    `KOGITO_CONSOLES_KEYCLOAK_UPDATE_TOKEN_VALIDITY`    |               
Update token validity in minutes.               | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
-   |      `KOGITO_CONSOLES_KEYCLOAK_HEALTH_CHECK_URL`       |                  
Keycloak health-check URL.                   | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
-   |            `KOGITO_CONSOLES_KEYCLOAK_REALM`            |                  
   Keycloak realm name.                      | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
-   |             `KOGITO_CONSOLES_KEYCLOAK_URL`             |                  
    Keycloak auth URL.                       | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
-   |          `KOGITO_CONSOLES_KEYCLOAK_CLIENT_ID`          |                  
    Keycloak Client ID.                      | See [ defaultEnvJson.ts 
](../runtime-tools-managment-console-webapp/build/defaultEnvJson.js) |
+   |                          Name                          |                  
        Description                          |                                  
          Default                                            |
+   | :----------------------------------------------------: | 
:-----------------------------------------------------------: | 
:-------------------------------------------------------------------------------------------:
 |
+   |   `RUNTIME_TOOLS_MANAGEMENT_CONSOLE_KOGITO_ENV_MODE`   | Env Mode: "PROD" 
or "DEV". PROD enables Keycloak integration. |                                  
          "PROD"                                             |
+   |   `RUNTIME_TOOLS_MANAGEMENT_CONSOLE_KOGITO_APP_NAME`   |                 
Management Console app name.                  | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |
+   | `RUNTIME_TOOLS_MANAGEMENT_CONSOLE_KOGITO_APP_VERSION`  |                
Management Console app version.                | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |
+   | `RUNTIME_TOOLS_MANAGEMENT_CONSOLE_DATA_INDEX_ENDPOINT` |        The URL 
that points to the Data Index service.         | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |
+   |    `KOGITO_CONSOLES_KEYCLOAK_DISABLE_HEALTH_CHECK`     |                
Disables Keycloak health-check.                | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |
+   |    `KOGITO_CONSOLES_KEYCLOAK_UPDATE_TOKEN_VALIDITY`    |               
Update token validity in minutes.               | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |
+   |      `KOGITO_CONSOLES_KEYCLOAK_HEALTH_CHECK_URL`       |                  
Keycloak health-check URL.                   | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |
+   |            `KOGITO_CONSOLES_KEYCLOAK_REALM`            |                  
   Keycloak realm name.                      | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |
+   |             `KOGITO_CONSOLES_KEYCLOAK_URL`             |                  
    Keycloak auth URL.                       | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |
+   |          `KOGITO_CONSOLES_KEYCLOAK_CLIENT_ID`          |                  
    Keycloak Client ID.                      | See [ defaultEnvJson.js 
](../runtime-tools-management-console-webapp/build/defaultEnvJson.js) |

Review Comment:
   Thanks for fixing the typo. Not sure I understand the change from `ts` to 
`js` though.



##########
packages/sonataflow-management-console-image-env/README.md:
##########
@@ -0,0 +1,48 @@
+<!--
+   Licensed to the Apache Software Foundation (ASF) under one
+   or more contributor license agreements.  See the NOTICE file
+   distributed with this work for additional information
+   regarding copyright ownership.  The ASF licenses this file
+   to you under the Apache License, Version 2.0 (the
+   "License"); you may not use this file except in compliance
+   with the License.  You may obtain a copy of the License at
+     http://www.apache.org/licenses/LICENSE-2.0
+   Unless required by applicable law or agreed to in writing,
+   software distributed under the License is distributed on an
+   "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+   KIND, either express or implied.  See the License for the
+   specific language governing permissions and limitations
+   under the License.
+-->
+
+## @kie-tools/sonataflow-management-console-image-env
+

Review Comment:
   Please describe this package in a few words, targeting users who may consume 
it.



##########
packages/sonataflow-mock-server/README.md:
##########
@@ -0,0 +1,11 @@
+# SonataFlow Mock Server
+
+### Description
+
+The SonataFlow Mock Server provides the SonataFlow GraphQL and OpenAPI 
endpoints.
+
+### Usage
+
+To run the development server, use the following command:
+
+`pnpm start`

Review Comment:
   Not sure I'm onboard with the name... What packages depend on it? Or is it a 
mock for a generic sonataflow app?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to