backdrop-contrib / paragraphs

Paragraphs module to control your content flow
https://backdropcms.org/project/paragraphs
GNU General Public License v2.0
5 stars 11 forks source link

Remove paragraphs_bundle table #157

Closed herbdool closed 12 months ago

herbdool commented 1 year ago

No longer needed now that the bundles are stored in config.

herbdool commented 1 year ago

I've got a PR @laryn.

There already was a line to remove paragraphs_bundle but it kept the schema. So I removed that and had it check for existence of the table for those upgrading from d7.

laryn commented 1 year ago

@herbdool Thanks for catching this. Do we need to move that code to drop the table into a separate update hook at this point, to catch the folks who installed Paragraphs on Backdrop and got an empty schema implemented?

herbdool commented 1 year ago

@laryn I updated the PR. Added another update hook to drop the table if it exists.

laryn commented 12 months ago

Thanks @herbdool -- tested locally and code looks good. Merged!