[WIP] Restructure build process #821

2018-12-14T08:22:22+01:00

skhameneh commented

2018-12-14 08:22:22 +01:00

(Migrated from github.com)

Prod npm run build && npm start
Dev npm run dev && npm start

Will be reducing dev to a single command.
Still working on productivity features.

Prod `npm run build && npm start` Dev `npm run dev && npm start` Will be reducing dev to a single command. Still working on productivity features.

skhameneh (Migrated from github.com) reviewed 2018-12-14 08:25:45 +01:00

server/models/index.js

					
				@ -64,1 +62,4 @@

				db['Trending'] = Trending(sequelize, Sequelize);

				db['User'] = User(sequelize, Sequelize);

				db['Views'] = Views(sequelize, Sequelize);

skhameneh (Migrated from github.com) commented

2018-12-14 08:24:07 +01:00

Had to remove sequelize.import because it attempts to read file runtime information that is not available when files are bundled.

Had to remove `sequelize.import` because it attempts to read file runtime information that is not available when files are bundled.

server/models/user.js

					
				@ -62,3 +62,3 @@

				  // pre-save hook method to hash the user's password before the user's info is saved to the db.

				  User.hook('beforeCreate', (user, options) => {

				  User.addHook('beforeCreate', (user, options) => {

				    logger.debug('User.beforeCreate hook...');

skhameneh (Migrated from github.com) commented

2018-12-14 08:24:31 +01:00

Updated legacy command

server/render/handleShowRender.jsx

					
				@ -0,0 +94,4 @@

				    // send the rendered page back to the client

				    res.send(renderFullPage(helmet, html, preloadedState));

				  };

skhameneh (Migrated from github.com) commented

2018-12-14 08:24:47 +01:00

Will remove

skhameneh (Migrated from github.com) reviewed 2018-12-14 08:49:45 +01:00

package.json

					
				@ -11,3 +8,4 @@

				    "dev": "webpack --config webpack.config.js --mode=development",

				    "configure": "node cli/configure.js",

				    "fix": "eslint . --fix",

				    "lint": "eslint .",

skhameneh (Migrated from github.com) commented

2018-12-14 08:49:39 +01:00

No more crazy multi-staged file pipelines!

Sign in to join this conversation.

No reviewers

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/spee.ch#821