Add a "Notes" textarea to each inject

I'll be starting to work on this issue now!

Small change, big difference! :)

Nice work @LRWebks you can remove the "Draft" prefix. I think it should only be used, if the changes are more or less experimental / unclear. Otherwise it prevents tests from running.

Code looks good to me. Perhaps you could add an "Internal" prefix to the "Notes" label => "Internal notes" to clarify that these aren't shown anywhere.

Could we also have a simple addition to tests to ensure it is saved as expected?

Then this LGTM and is ready for final review :)
Nice addition for daily work, I would have needed this several times already.

I'll be implementing your proposed changes quickly.

Done! Surely, a test of the Form Save behavior for the notes would be necessary, however this test does not exist in this code version but is being implemented in ✨ Add "status" option to allow enabling / disabling snippets Needs work , so it's the same thing as with the unrelated change from before. We just shouldn't forget about this once both issues are merged!

Thanks @LRWebks! LGTM!

So settings this RTBC, but whoever merges the other issue should please then set this NW to implement the tests here finally. Once it goes green, this can be set RTBC again and be merged! :)

✨ Add "status" option to allow enabling / disabling snippets Needs work

I feel like this is an unnecessary bloat to the already complex form. And comments can often be placed directly in the JS/CSS asset. If there is a desire for notes field, it would be straight forward to modify the entity type to add the field.

Thanks for the feedback @pookmish. I'm a bit disappointed to no see this merged, because we had many cases where that description would have helped our team and our clients a lot.

Of course, you can use the workaround to put the comments into code or write a custom module, but we ran into this in different projects, so I'd see this as a generally helpful feature. We can definitely make it less visible if that helps.

In our case, it's often additional information for the site builders / administrator that needs to be placed besides the technical script. For example, what it is used for, in which areas, which GDPR terms apply here etc.

Of course, I accept, if you still decline the feature.