- Issue created by @catch
- First commit to issue fork.
- Merge request !6868Issue #3425105: Don't use text_with_summary in Umami → (Closed) created by smustgrave
- Status changed to Needs review
9 months ago 7:53pm 2 March 2024 - 🇺🇸United States smustgrave
Think I found the issue. Had to use a new field name vs just "body"
- 🇺🇸United States smustgrave
Actually had to change a lot. This the route we will have to go other places since the "body" field is hardset to be text_with_summary.
- 🇮🇪Ireland markconroy
I think this looks like a great idea, but now I also have a worry about us having a field called
body
and a field calledfield_body
if someone tries to add a new field to a content type and uses the "reuse existing field" approach. Also if you create a new content type it will automatically get thebody
field added.I guess people don't need to be creating new fields and content types, since this is a fully formed demo.
I wonder could we disable the default
body
field from the UI fully? - 🇬🇧United Kingdom catch
@markconroy there's a bit of a chicken and egg situation between this issue and other child issues of 🌱 Deprecate text_with_summary Closed: duplicate .
If we eventually remove the hard-coding of the 'body' field and make it uninstallable, then Umami would just be able to ignore it entirely.
If we want to keep the hard-coding of the 'body' field, then it might make sense to switch it to a standard text field for new installs, in which case this MR might not need to change anything.
- Status changed to Postponed
9 months ago 1:52pm 11 March 2024 - 🇺🇸United States smustgrave
Opened a META to track decisions 🌱 [Meta] Deprecate text_with_summary Active
I'm postponing this till we decide what to do with the body field.
- Status changed to Needs review
7 months ago 10:04pm 22 May 2024 - Status changed to RTBC
6 months ago 1:40pm 14 June 2024 - 🇮🇪Ireland markconroy
- 🇬🇧United Kingdom catch
Had another look through this. I think the confusion between body vs. field_body is something a lot of real sites already have, and going to be less confusing than having the text summary widget which then never gets used. If we change how the body field works, we could eventually rename this again back to 'body' if we want to, or just leave it as is without the dormant body field.
Committed/pushed to 11.x, thanks!
- Status changed to Fixed
6 months ago 1:31pm 18 June 2024 Automatically closed - issue fixed for 2 weeks with no activity.