- 🇸🇰Slovakia coaston
#44 worked for me, But without #46 I would not notice it, so thank you @lexsoft.
- First commit to issue fork.
- Merge request !118Issue #2877695: Skip saving empty paragraphs for certain types → (Open) created by codebymikey
- 🇨🇳China m0925j
We still have this problem in drupal version 10.2.6, paragraph version 8.x-1.17, what I want is for the default to not create an empty paragraph entity when no value is filled in, for the default to display the paragraph widget in the node creation form, and for the default to not create an empty paragraph entity when the Add paragraph button is clicked but no value is actually filled in. Instead of having to be manually removed by the user.
- 🇦🇷Argentina abelpzl
I ran into the same problem on drupal 10.3.1 and paragraphs 1.17.
The merge request made by Codebymikey solved my problem. - Status changed to RTBC
7 months ago 7:52am 2 August 2024 - 🇮🇳India diwakar07
Hi,
I reviewed MR !118 on drupal 10.3.0 and paragraphs 1.17, it successfully fixes the issue.
It adds an option in the paragraph type edit form to Enable/Disable
Save paragraph instances, even when fields are empty
, by default it is enabled.
On disabling this option, I was able to save the node by adding a blank paragraph type, and no label was rendered for the blank paragraph type added on node.
Attached are the SS for references.This looks good to me.
Moving to RTBC!Thanks.
- Status changed to Needs work
7 months ago 12:19pm 6 August 2024 - 🇮🇳India sourav_paul Kolkata
I've checked the issue On D10.3.3.
MR!118 was applied cleanly .MR fixed the issue.
Attaching SS:
Before applying the MR:
After applying the MR:
- 🇮🇳India sourav_paul Kolkata
I've resolved the review changes..
Please Review the updated changes.
Updated the PR and requesting further review of the changes.
Would ideally be nice if a test was also introduced as part of this change to avoid unintentionally breaking it.
I don't have time to write a functional test for it at the moment, but if someone else wants to have a go at it, you're more than free to!
- Status changed to Needs review
4 months ago 9:14am 22 October 2024 codebymikey → changed the visibility of the branch skip to hidden.
- 🇺🇦Ukraine voleger Ukraine, Rivne
Uploading the MR 118 diff as a patch for composer patching needs