- Issue created by @David Fiaty
- Assigned to paulrad
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 9:07am 25 September 2023 - 🇺🇦Ukraine paulrad
I've tested and the module works as expected.
@David Fiaty, please go to theadmin/config/workflow/publishcontentand enable 'Publish and unpublish via checkbox' if you want to have this possibility.But it could be great to have this option enabled by default, so I'm making a merge request for maintainers to decide.
- 🇬🇧United Kingdom David Fiaty
The suggestion works indeed. However I believe a freshly installed module shouldn't change any core Drupal config parameter without user action. Great work and thank you very much for the quick reply.
- Status changed to Postponed: needs info
4 months ago 8:48am 13 August 2024 - 🇪🇸Spain rodrigoaguilera Barcelona
This still happens on Drupal 10/11 with the latest version of the module?
- Status changed to RTBC
4 months ago 9:41am 13 August 2024 - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
I don't really see an immediate cause of why uninstalling this module would delete views.
I do follow the idea of setting the checkbox enabled by default though, so things work like they did before, until you choose to disable the option.
- 🇪🇸Spain rodrigoaguilera Barcelona
Ok.
Then the proper place to set defaults would be
https://git.drupalcode.org/project/publishcontent/-/blob/8.x-1.x/config/...The update hook can get the update too but it is not very relevant.
- Status changed to Needs work
4 months ago 10:46am 13 August 2024 - First commit to issue fork.
- 🇬🇧United Kingdom nexusnovaz
Hey, updated
publishcontent.settings.ymlto make ui_checkboxes true. Should the change to the .install be reverted back to 0?Will mark as needs review in case this is all good. Thanks
- 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Updating an existing update hook won't have any effect on active installs. Not sure we really need to create a new update hook for this
