strapi icon indicating copy to clipboard operation
strapi copied to clipboard

[DEITS] add the cli backbone

Open innerdvations opened this issue 2 years ago • 1 comments

What does it do?

  • Adds the export command (works in theory, but transfer engine isn't implemented yet)
  • Adds the import command (no idea if it works, but the backbone is there and it builds)

Why is it needed?

So the export code can be run from the command line

How to test it?

Run yarn strapi export --help and you will see help for the command. Run yarn strapi export and you will get an error that no file was created (because the export code isn't implemented up yet)

Related issue(s)/PR(s)

innerdvations avatar Oct 17 '22 07:10 innerdvations

Codecov Report

Base: 58.75% // Head: 62.45% // Increases project coverage by +3.70% :tada:

Coverage data is based on head (282c70a) compared to base (d413c55). Patch has no changes to coverable lines.

:exclamation: Current head 282c70a differs from pull request most recent head c98c3e9. Consider uploading reports for the commit c98c3e9 to get more accurate results

Additional details and impacted files
@@                Coverage Diff                 @@
##           features/deits   #14636      +/-   ##
==================================================
+ Coverage           58.75%   62.45%   +3.70%     
==================================================
  Files                1322     1043     -279     
  Lines               32026    22336    -9690     
  Branches             5968     3860    -2108     
==================================================
- Hits                18816    13950    -4866     
+ Misses              11349     7372    -3977     
+ Partials             1861     1014     -847     
Flag Coverage Δ
front 62.45% <ø> (ø)
unit ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...utils/typescript/lib/generators/schemas/mappers.js
...ackages/core/admin/server/validation/permission.js
...b/services/entity-service/attributes/transforms.js
packages/core/strapi/lib/commands/admin-create.js
...ges/core/admin/server/services/project-settings.js
...ackages/core/strapi/lib/services/metrics/sender.js
...ils/typescript/lib/utils/resolve-config-options.js
...e/permissions/lib/engine/abilities/casl-ability.js
...ages/plugins/i18n/server/migrations/field/utils.js
packages/core/admin/ee/server/services/passport.js
... and 269 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Oct 17 '22 07:10 codecov[bot]

Yarn.lock doesn't have any changes. The error seems to be that @strapi/data-transfer is missing, which is odd because I'm not getting that message locally and it is included in package.json for the command line lib. Think it could it have something to do with data-transfer being in typescript?

innerdvations avatar Oct 24 '22 13:10 innerdvations

@Convly I just checked. It works locally only because the dist folder exists. This is another hurdle to overcome with including typescript in the monorepo.

innerdvations avatar Oct 24 '22 14:10 innerdvations