[BUG] Environment: undefined in migration CLI summary

deluan / contentful-migrate

🐎 Schema migration tooling for Contentful, with state management

https://www.contentful.com/blog/2018/09/13/content-model-changes-scale-telus-cms-as-code/

MIT License

60 stars 37 forks source link

[BUG] Environment: undefined in migration CLI summary #180

Closed parkerhutchinson closed 3 years ago

parkerhutchinson commented 3 years ago

Description

Noticed that when running migrations that the Environment chalk label was returning undefined.

CleanShot 2020-12-27 at 23 11 54@2x

Solution

adding environmentId to renderPlan() on the renderMigration async call. renderPlan() expects the environmentId be set when using it. since we know environmentId is either CONTENTFUL_ENV_ID env variable or "master" its safe to pass directly here.

parkerhutchinson commented 3 years ago

@deluan any chance we can get this merged in? this is actually kinda a serious bug that is easily fixed.

deluan commented 3 years ago

Sorry for the delay! Will release a new version later today