#Needs subsystem maintainer review

rbrissaud → changed the visibility of the branch 2842780-11.x to hidden.

rbrissaud → changed the visibility of the branch 2842780-11.x to hidden.

rbrissaud → changed the visibility of the branch drupal-2842780-10.4.x-add-token-site-logo to hidden.

rbrissaud → changed the visibility of the branch 2842780-11.x to hidden.

rbrissaud → changed the visibility of the branch drupal-2842780-10.4.x-add-token-site-logo to hidden.

rbrissaud → changed the visibility of the branch drupal-2842780-10.4.x-add-token-site-logo to hidden.

rbrissaud → changed the visibility of the branch drupal-2842780-10.4.x-add-token-site-logo to hidden.

📌 Consider falling back to webp based on filesize Active has a possible approach to the trade-offs between AVIF and webp for filesize.

No worries @penyaskito 😁 you're not the only one who has been misleaded by unresolved threads. I don't really know what it depends on. Sometimes I see the "Resolve thread" button at GitLab, but sometimes not.

OK, I'll stop messing around. It's already fixed, just we can't resolve threads.

NW per https://git.drupalcode.org/project/drupal/-/merge_requests/11220#note_49.... We should accept that suggestion, then IMHO we should be good to go.

Since the changes look good to me and the tests pass, I believe it's safe to switch back to RTBC.

Thanks @godotislate!

Added CR: https://www.drupal.org/node/3519307 →

I added a couple minor comments on the MR after looking again while writing the CR.

We need a change record to detail the addition of StateInterface::getValuesSetDuringRequest() - https://www.drupal.org/node/add/changenotice?field_project=3060&field_is... →

Just following up if additional steps can be provided, else we could probably close this one out.

As per https://www.drupal.org/project/drupal/issues/2709387#comment-16039612 🐛 Node creation date is populated on creation form Needs work

hook_entity_presave could be used to implement this in your custom module if needed.