feat!: initProps concept removed from ILC Registry

registry/client/src/appRoutes/Edit.js

@@ -65,7 +65,7 @@ const InputForm = ({mode = 'edit', ...props}) => {
                             { id: 'essential', name: 'Essential' },
                             { id: 'regular', name: 'Regular' },
                         ]} />
-                        <JsonField source="props" label="App props"/>
+                        <JsonField source="props" label="Properties that will be passed to application at current route"/>
                     </SimpleFormIterator>
                 </ArrayInput>
             </FormTab>

registry/client/src/apps/Edit.js

@@ -54,8 +54,7 @@ const InputForm = ({mode = 'edit', ...props}) => {
                 <NumberInput source="ssr.timeout" label="Request timeout, in ms" />
             </FormTab>
             <FormTab label="Props">
-                <JsonField source="props"/>
-                <JsonField source="initProps"/>
+                <JsonField source="props" label="Properties that will be passed to application"/>
             </FormTab>
         </TabbedForm>
     );

registry/client/src/apps/dataTransform.js

@@ -9,18 +9,12 @@ export function transformGet(app) {
     if (app.props) {
         app.props = JSON.stringify(app.props);
     }
-    if (app.initProps) {
-        app.initProps = JSON.stringify(app.initProps);
-    }
 }
 
 export function transformSet(app) {
     if (app.props) {
         app.props = JSON.parse(app.props);
     }
-    if (app.initProps) {
-        app.initProps = JSON.parse(app.initProps);
-    }
     if (app.dependencies) {
         app.dependencies = app.dependencies.reduce((acc, v) => {
             acc[v.key] = v.value;

registry/server/apps/interfaces/index.ts

@@ -9,7 +9,6 @@ export default interface App {
     props?: string, // JSON({ [propName: string]: any })
     configSelector?: string,
     ssr: string, // JSON({ src: string, timeout: number })
-    initProps?: string, // JSON({ [propName: string]: any })
 }
 
 export const appNameSchema = Joi.string().trim().min(1);
@@ -25,7 +24,6 @@ const commonApp = {
         src: Joi.string().trim().uri().required(),
         timeout: Joi.number().required(),
     }),
-    initProps: Joi.object().default({}),
     kind: Joi.string().valid('primary', 'essential', 'regular'),
 };
 

registry/server/apps/routes/createApp.ts

@@ -22,7 +22,7 @@ const validateRequestBeforeCreateApp = validateRequestFactory([{
 const createApp = async (req: Request, res: Response): Promise<void> => {
     const app = req.body;
 
-    await db('apps').insert(stringifyJSON(['dependencies', 'props', 'ssr', 'initProps', 'configSelector'], app));
+    await db('apps').insert(stringifyJSON(['dependencies', 'props', 'ssr', 'configSelector'], app));
 
     const [savedApp] = await db.select().from<App>('apps').where('name', app.name);
 

registry/server/apps/routes/updateApp.ts

@@ -43,7 +43,7 @@ const updateApp = async (req: Request<UpdateAppRequestParams>, res: Response): P
         return;
     }
 
-    await db('apps').where({ name: appName }).update(stringifyJSON(['dependencies', 'props', 'ssr', 'initProps', 'configSelector'], app));
+    await db('apps').where({ name: appName }).update(stringifyJSON(['dependencies', 'props', 'ssr', 'configSelector'], app));
 
     const [updatedApp] = await db.select().from<App>('apps').where('name', appName);
 

registry/server/migrations/20200518134531_apps_initProps_removal.ts

@@ -0,0 +1,16 @@
+import * as Knex from "knex";
+
+
+export async function up(knex: Knex): Promise<any> {
+    return knex.schema.table('apps', function (table) {
+        table.dropColumn('initProps');
+    })
+}
+
+
+export async function down(knex: Knex): Promise<any> {
+    return knex.schema.createTable('apps', table => {
+        table.json('initProps');
+    });
+}
+

registry/server/routes/config.ts

@@ -25,7 +25,6 @@ router.get('/', async (req, res) => {
     data.apps = apps.reduce((acc, v) => {
         v.ssr = JSON.parse(v.ssr);
         v.dependencies = JSON.parse(v.dependencies);
-        v.initProps = JSON.parse(v.initProps);
         v.props = JSON.parse(v.props);
         if (sharedProps.length && v.configSelector !== null) {
             JSON.parse(v.configSelector).forEach((configSelectorName: string) => {

registry/server/seeds/01_apps.ts

@@ -15,7 +15,6 @@ export async function seed(knex: Knex): Promise<any> {
                 src: 'http://localhost:8235/',
                 timeout: 1000,
             }),
-            initProps: '{}',
             props: '{}',
             kind: 'essential',
         }, {
@@ -27,7 +26,6 @@ export async function seed(knex: Knex): Promise<any> {
                 rxjs: 'https://unpkg.com/[email protected]/bundles/rxjs.umd.js',
                 '@portal/fetchWithCache': `http://${publicHost}:8238/fetchWithCache.js`,
             }),
-            initProps: '{}',
             props: JSON.stringify({
                 publicPath: `http://${publicHost}:8236/`
             }),
@@ -38,7 +36,6 @@ export async function seed(knex: Knex): Promise<any> {
             dependencies: JSON.stringify({
                 '@portal/fetchWithCache': `http://${publicHost}:8238/fetchWithCache.js`,
             }),
-            initProps: '{}',
             props: '{}',
             kind: 'primary',
         }, {
@@ -51,9 +48,6 @@ export async function seed(knex: Knex): Promise<any> {
             }),
             assetsDiscoveryUrl: 'http://127.0.0.1:8239/_spa/dev/assets-discovery',
             dependencies: '{}',
-            initProps: JSON.stringify({
-                publicPath: `http://${publicHost}:8239/dist/`
-            }),
             props: JSON.stringify({
                 publicPath: `http://${publicHost}:8239/dist/`
             }),
@@ -66,14 +60,12 @@ export async function seed(knex: Knex): Promise<any> {
                 timeout: 1000,
             }),
             dependencies: '{}',
-            initProps: '{}',
             props: '{}',
             kind: 'primary',
         }, {
             name: '@portal/fetchWithCache',
             spaBundle: `http://${publicHost}:8238/fetchWithCache.js`,
             dependencies: '{}',
-            initProps: '{}',
             props: '{}',
             kind: 'essential',
         },

registry/tests/apps.spec.ts

@@ -16,7 +16,6 @@ const example = <any>{
         kind: 'primary',
         // dependencies: {},
         // props: {},
-        // initProps: {},
     }),
     updated: Object.freeze({
         name: '@portal/ncTestAppReactssr',
@@ -38,9 +37,6 @@ const example = <any>{
             assetsPath: 'http://127.0.0.1:3001/uisamplereactUpdated',
             locationStrategy: 'browserHistoryUpdated',
         },
-        initProps: {
-            assetsPath: 'http://127.0.0.1:3001/uisamplereact',
-        },
     }),
 };
 example.encodedName = encodeURIComponent(example.correct.name);
@@ -65,7 +61,6 @@ describe(`Tests ${example.url}`, () => {
                 assetsDiscoveryUrl: 789,
                 dependencies: 456,
                 props: 789,
-                initProps: 456,
             };
 
             let response = await request.post(example.url)
@@ -82,7 +77,6 @@ describe(`Tests ${example.url}`, () => {
                 '"props" must be of type object\n' +
                 '"configSelector" must be an array\n' +
                 '"ssr" must be of type object\n' +
-                '"initProps" must be of type object\n' +
                 '"name" must be a string'
                 );
 
@@ -193,7 +187,6 @@ describe(`Tests ${example.url}`, () => {
                 assetsDiscoveryUrl: 789,
                 dependencies: 456,
                 props: 789,
-                initProps: 456,
                 kind: 'origin',
             };
 
@@ -211,7 +204,6 @@ describe(`Tests ${example.url}`, () => {
                 '"props" must be of type object\n' +
                 '"configSelector" must be an array\n' +
                 '"ssr" must be of type object\n' +
-                '"initProps" must be of type object\n' +
                 '"kind" must be one of [primary, essential, regular]'
             );
             expect(response.body).deep.equal({});