Jump to conversation
Unresolved conversations (0)
Nice work!

Nice work!

All of your conversations have been resolved.

Resolved conversations (6)
@alexandrebodin alexandrebodin Dec 11, 2020
You should just wrap the resolve in a try catch because if strapi fails for another reason on load it won't be displayed
...ssets/migrations/scripts/migrate-3.4.0.js
@alexandrebodin alexandrebodin Dec 7, 2020
are you sure it works ? looks like the options should be an object with { paths: ['aabc' ] } from the doc
Outdated
...ssets/migrations/scripts/migrate-3.4.0.js
@alexandrebodin alexandrebodin Dec 4, 2020
use a setter in case the field would not be here for some reason ?
Outdated
...ssets/migrations/scripts/migrate-3.4.0.js
@alexandrebodin alexandrebodin Dec 3, 2020
why not do a simple require('strapi') ?
Outdated
...ssets/migrations/scripts/migrate-3.4.0.js
alexandrebodin petersg83
Alexandre BODIN and Pierre Noël
@derrickmehaffy derrickmehaffy Dec 3, 2020
Would it be better to link to the GitHub script instead or do you think it's better to include this in the docs?
...n-guide/migration-guide-3.3.x-to-3.4.0.md
derrickmehaffy petersg83
DMehaffy and Pierre Noël
@derrickmehaffy derrickmehaffy Dec 3, 2020
Could add another step to delete the script (optional)
...n-guide/migration-guide-3.3.x-to-3.4.0.md