- Issue created by @lauriii
- Status changed to Needs review
over 1 year ago 8:36am 17 June 2023 - last update
over 1 year ago 29,485 pass, 1 fail The last submitted patch, 2: 3367420-2.patch, failed testing. View results →
- last update
over 1 year ago 29,485 pass, 1 fail - Status changed to Needs work
over 1 year ago 1:10pm 17 June 2023 - 🇫🇷France nod_ Lille
Because there is no more ajax involved I'd probably listen directly to the change and input event for machine names instead of relying on the the formUpdated event.
- Status changed to Needs review
over 1 year ago 1:32pm 17 June 2023 - last update
over 1 year ago 29,499 pass - Status changed to RTBC
over 1 year ago 4:21pm 17 June 2023 - 🇺🇸United States smustgrave
No failures is always good.
Applied the patch and tested by creating a content type and the machine name field updated as I typed.
So is next step to make these faster to generate them before I open my laptop? haha
- Status changed to Needs work
over 1 year ago 11:02pm 17 June 2023 - 🇫🇷France nod_ Lille
minor thing but could be useful to add a comment to say why we're not using formupdated and using change/input is fine since there is no ajax request involved
- Status changed to RTBC
over 1 year ago 8:50am 18 June 2023 - last update
over 1 year ago 29,499 pass - 🇫🇮Finland lauriii Finland
Good idea 👍 Moving back to RTBC since the interdiff is just for a simple inline code comment. Feel free to adjust the comment if you think it should be worded differently. 😊
- Status changed to Needs work
over 1 year ago 7:58am 19 June 2023 - 🇫🇷France nod_ Lille
almost there. We need to keep the event namespace since it's used to remove event listeners when clicking on the edit button. Right now when clicking edit and changing the field name, the machine name is also updated.
- Status changed to Needs review
over 1 year ago 8:08am 19 June 2023 - last update
over 1 year ago CI aborted - 🇫🇮Finland lauriii Finland
Good catch 👍 I searched for the whole event with the namespace but not for just the namespace 🙈
- last update
over 1 year ago 29,499 pass - Status changed to Fixed
over 1 year ago 8:38am 19 June 2023 Automatically closed - issue fixed for 2 weeks with no activity.