Fix and update the cli #847

lyzhovnik · 2018-09-11T13:12:34Z

This pull request continues works from the #790 branch, but does not have crud cli.
Was made the following progress:

fixed bug according to the issue CLI Error #846;
refactored all the code of the cli tool to reach more flexibility and reliability;
updated templates to the latest structure of the Kit;
converted templates into Typescript.

Also, Thank @Werter12 for the great work on the cli.

…ating for addCrud

…checks

…config

…me variables. Change comments

…ule functions

…ions

# Conflicts: # tools/cli/module.js # yarn.lock

Fixes: #846

larixer

Looks good to me, thank you Nikita!

Fixes: #847

Werter12 and others added 30 commits July 26, 2018 12:44

Add cli-crud core functionality

ca84bde

Add generated containers file

fe7cd96

Add domain-schema dependencies

b9a5fad

Fix pagiantion: wrong __typename being generated

a7a7630

Create commandInvoker, split addCrud and addModule

a380e0c

Add runCommand to CommandInvoker and change require in addCrud

5921d22

Update addModule index.js file changing, get rid of client module cre…

dc7f9ed

…ating for addCrud

Refactor addModule. Reduce read and write file operations

88c7731

Fix tablePrefix generating. Divide renameFiles function

dc427ae

Init delete module command

c5ee744

Refactor deleteModule. Reduce read and write file operations and add …

a591425

…checks

Add domain-schema/knex dependency

4798deb

Add reading index file no matter what extension it has

577e2b6

Add absent types to graphql template

c9d5087

Fix babel class compilation issue, after running migrate command

2a6d4ad

Add -m option for deletemodule command to delete migration and seeds

ec74a4b

Add sub command deleteMigrations. Add config for repeated paths

efe807b

Add sub command add migration. Replace hardcoded paths on paths from …

6f37efd

…config

Replace paths in deleteMigrations function. Delete console.logs. Rena…

91cb235

…me variables. Change comments

Delete unnecessary files. Make small changes to addCrud and deleteMod…

9185223

…ule functions

Fix base path

02067b9

Add new dependencies

2f501e6

Add crud and helpers from cli-crud

6364aa7

Refactor selectBy and getSelectFields functions

9d9af2b

Refactor getList ordering and filtering, getPaginated, getTotal funct…

f77ccb8

…ions

Merge branch 'master' into add-delete-crud-cli-server-database

7533456

# Conflicts: # tools/cli/module.js # yarn.lock

Resolve version conflicts in yarn.lock for shelljs lib

ee04e55

Fixes: #846

Update cli templates to the latest structure

2d404c6

Remove crud cli

c7d8161

Change file extensions of templates to .ts

e1ee381

lyzhovnik added 3 commits September 11, 2018 14:44

Convert templates into Typescript

4c621ef

Add internalization to the templates

d406dca

Merge branch 'master' into fix/update-cli

00a84d2

lyzhovnik requested a review from Werter12 September 11, 2018 13:16

Remove crud dependencies

de62fc4

larixer approved these changes Sep 11, 2018

View reviewed changes

larixer merged commit b1180f7 into master Sep 11, 2018

larixer deleted the fix/update-cli branch September 11, 2018 16:44

This was referenced Sep 11, 2018

CLI Error #846

Closed

Error in client../../locales/index.js #852

Closed

lyzhovnik mentioned this pull request Sep 25, 2018

pre-commit hook fail on template graphql with addmodule #523

Closed

larixer pushed a commit that referenced this pull request Oct 23, 2018

Fix and update the cli (#847)

fa56153

Fixes: #847

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix and update the cli #847

Fix and update the cli #847

lyzhovnik commented Sep 11, 2018 •

edited

Loading

larixer left a comment

Fix and update the cli #847

Fix and update the cli #847

Conversation

lyzhovnik commented Sep 11, 2018 • edited Loading

larixer left a comment

Choose a reason for hiding this comment

lyzhovnik commented Sep 11, 2018 •

edited

Loading