search

bolt / core

🧿 Bolt 5 core
https://boltcms.io
MIT License
550 stars 161 forks source link

Add YAML Migrations #3497

Closed xiaohutai closed 11 months ago

xiaohutai commented 1 year ago

See for the diff: https://gist.github.com/xiaohutai/b5f44fec671cfff6691255354f6f769b

Not sure if the uncommented canonical: https://example.org was intentional.

Need to test the integrations before merging in.

bobdenotter commented 1 year ago

I think JSON_VALUE needs to be added:

https://github.com/bolt/project/blob/5.2-php-8/config/packages/doctrine.yaml#L37

And also: adapter: cache.system -> adapter: cache.app

https://github.com/bolt/project/commit/f33248d487b5b6a09a38224f890d259f9ea07003#diff-79c1dc283ac8f8d1d02991db661137a22a20b13cd277962eae1d0dbfb2d96467

xiaohutai commented 1 year ago

I found the one for for doctrine.yaml: https://github.com/bolt/core/issues/2088

Not sure if 5.1.6 is still the correct version here, but it seems the existing config in 5.1 already has it added: https://github.com/bolt/core/blob/5.1/config/packages/doctrine.yaml#L35C2-L35C21 (Ideally this migration should've been added with the config change 😋)

I've added those, but the bolt/core doesn't have the cache.app so I changed it in this commit as well.