- Issue created by @Liam Morland
- Merge request !16Issue #3501817: Removed extra space addition before or after seperator. โ (Merged) created by sayan_k_dutta
- ๐ฎ๐ณIndia sayan_k_dutta
Made the changes. Please review.
Also the pipeline was failing as the tests module was incompatible with D11. Made the fix for it. Pipeline is green now. - ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
Thanks for the patch.
It seems to me that the default
separator
should stay the same so that existing installs do not change. - Assigned to koustav_mondal
- Status changed to Needs review
2 months ago 11:29am 31 March 2025 - ๐ฎ๐ณIndia koustav_mondal Kolkata
Installed the module and tested. Looks good. Attaching screenshot for reference.
- First commit to issue fork.
-
astonvictor โ
committed 95ac11b7 on 8.x-1.x authored by
sayan_k_dutta โ
Issue #3501817: Removed extra space addition before or after seperator.
-
astonvictor โ
committed 95ac11b7 on 8.x-1.x authored by
sayan_k_dutta โ
-
astonvictor โ
committed 67b00c99 on 8.x-1.x
Issue #3501817 - Fix tests
-
astonvictor โ
committed 67b00c99 on 8.x-1.x