Display the deployment_identifier on the status page

https://www.drupal.org/project/dis →

Re: #2 this contrib module exists, but seems like a trivial update to the status page that shouldn't require a contrib module to achieve.

Thanks for raising this issue Kim.

But why only display? To me the warning when identifier is missing is the most important part of the DIS module, and I clearly failed to express this on the module project page and in the module messages.

I consider having deployment identifier set and changed with every version of the application being deployed is the best practice that community should follow and this module was my attempt to spread the word.

A lot of sites wouldn't need to use a deployment identifier. I don't think we should make it a warning. This might be something we could keep the contrib module for if you want this behaviour.

Applied the MR and uncommented the line in settings.php

Verified I see the identifier on the status page

@RoSk0 if this gets into core, you could always do a hook_requirements_alter() and add a warning if there is no value set.

About #6 Kim, how do you decide which website need that or not?

I saw bug reports for broken updates because contrib modules was changing service signatures, but never faced them myself. I'm convinced that deployment identifier is the answer to that.

Not arguing about getting this into core for sure - let's have this committed!

I think we could consider using ✨ Create a new component: Toggletip RTBC to add information on where this can be configured if it's not set - as extra info rather than a warning? Tagging for follow-up to discuss that more.

Fixed the capitalisation issue on commit instead of holding this up further. As someone who uses deployment identifier in production I think this is a good idea and will only show on sites that use it anyway.

Committed and pushed 82032279d1 to 11.x and 47c383dd82 to 10.2.x. Thanks!

Feel free to open a followup to add a tooltip etc if you think this is a good idea.

Automatically closed - issue fixed for 2 weeks with no activity.

I created the followup, 📌 Add a toggletip for deployment identifier information Active