- 🇨🇭Switzerland vrancje Bern
This works perfectly on 10.0.11! Thanks for the work.
Are there any plans to reopen #3222990 → now that allowed formats is in 10.1? - 🇺🇸United States superfluousapostrophe
Something was updated in the 8.x-2.x-dev branch today and the patch in #32 is now broken. Any chance the patch can be re-rolled?
- Status changed to Needs work
8 months ago 7:49pm 6 May 2024 - First commit to issue fork.
- 🇮🇳India immaculatexavier
immaculatexavier → changed the visibility of the branch 3076448- to hidden.
- Status changed to Needs review
7 months ago 10:00am 23 May 2024 - last update
7 months ago 3 pass - 🇺🇸United States superfluousapostrophe
Thanks for the reroll! The #38 patch is now applying cleanly to the latest from the 2.x dev branch.
- Status changed to RTBC
5 months ago 9:35am 6 August 2024 - 🇮🇳India Tirupati_Singh
Hi, I tried applying the MR !4 as a patch and got error while applying the patch.
I applied the patch provided in comment #38 ✨ WYSIWYG support on each table cell for better content experience Needs review and it applied with no errors. After applying the patch<a href="https://www.drupal.org/files/issues/2024-05-23/wysiwyg-support-3076448-38.patch">wysiwyg-support-3076448-38.patch</a>
the raise feature request functionality is working as expected. I am attaching the before and after screenshots for reference. As patch #38 resolves the issue hence, moving the issue status to RTBC.As mentioned in comment #38 the provided patch can be converted into MR for the issue.
- 🇮🇳India gg24
Re-rolled the patch with the latest stable module release. Please review.
In #41 patch not working for existing field, so fixed the issue by adding missing required code for existing field & uploaded the patch
- 🇮🇩Indonesia awset
Patch #38 works perfectly in my case with the version 2.5-rc1.
DEV version dropped support for D10.2, where I still need it for now.However, I found a bug in #38 when doing an import via CSV, it is not populating the data.
I have fix the issue here, I also added the interdiff to make it easier to review. - 🇮🇳India Tirupati_Singh
Hi @awset, I've applied the provided wysiwyg-support-3076448-43.patch → in #43 ✨ WYSIWYG support on each table cell for better content experience Needs review , it applied cleanly with no errors. After applying, each table cells have user input format option. Now user can change the input format for table cells. The changes are working fine while using the module. However, after applying the patch, export table data as CSV feature is impacted due to the new changes. Previously the table data was exported as the actual data/value but after applying the patch table data is being stored as simply '
Array
' text instead of array. And when the table is being imported the table created is simply containing the text 'Array
' instead of actual data due to the incorrect data present in the CSV file. Previously the import and export table data as CSV was working fine. I'm attaching the screenshots of the before and after fixes for your reference. As the module functionality is impacted due to the latest patch, hence moving back the issue status to Needs work.Thanks!