- Issue created by @vakulrai
- Merge request !451Issue #3504452: Automators: The Joiner field setting is not considering the #default_value. → (Merged) created by vakulrai
- 🇮🇳India vakulrai
Created a MR to fix the issue , i can see the default value is being considered on the settings form.
Please review ! - 🇩🇪Germany marcus_johansson
Copy and paste buf caused by me :) Thanks @vakulrai - I added a comment on how I think you should fix it.
- 🇮🇳India vakulrai
Thank you @marcus_johansson for a review andpointing me to the code.
I have updated the MR accordingly and found one more issue about default value not being passed to "Other Joiner" select type, so i have fixed that in the same mr.
Hope its fine :)
-
marcus_johansson →
committed 226edb4a on 1.1.x
Issue #3504452: Automators: The Joiner field setting is not considering...
-
marcus_johansson →
committed 226edb4a on 1.1.x
-
marcus_johansson →
committed 226edb4a on 1.0.x
Issue #3504452: Automators: The Joiner field setting is not considering...
-
marcus_johansson →
committed 226edb4a on 1.0.x
-
marcus_johansson →
committed 226edb4a on 3479388-research-normalize-function-with-actions
Issue #3504452: Automators: The Joiner field setting is not considering...
-
marcus_johansson →
committed 226edb4a on 3479388-research-normalize-function-with-actions