🇦🇺Australia @peterwcm

Thank you @anybody :) I have removed unrelated changes and kept it simple. I will continue to do more testing from my end.

peterwcm → changed the visibility of the branch 3249822-translations-checksum-2.1 to hidden.

peterwcm → changed the visibility of the branch 3249822-translations-checksum-1.x to hidden.

peterwcm → changed the visibility of the branch 3249822-translations-checksum-2.x to hidden.

peterwcm → changed the visibility of the branch 2.x to hidden.

My site encountered this error after upgrading from 8.x-4.2 to 5.0.0-alpha3
My setup uses Smart IP and Smart IP MaxMind GeoIP2 binary database modules

Patch #5 works for me

Drupal version: 9.5.11
PHP: 8.1.29

Let's get a new 2.x version that is D10 compatible and drop D8 support from there. For 1.x, let's leave that in D8/D9.

Hi @linhnm,

I think it's a valid request to add in the media library form. However, the module itself is just to provide a Media source plugin, it shouldn't take care of the actual media type creation including all the field-related configs. It's up to sites to decide how they want to create or name the media type.

Checking upon some of the famous media source modules, they don't provide any default field or display configs for the same reasons.
https://www.drupal.org/project/media_entity_facebook →
https://www.drupal.org/project/media_entity_instagram →

Regards

Peter

A new release with D10 support has been released: https://www.drupal.org/project/smartip_language_detection/releases/2.0.0... →

peterwcm → made their first commit to this issue’s fork.

8.x is not going to be D10 compatible

Hi @jeroent

I am looking at the issues and will prepare for a new 2.x release but will probably drop D8 support going forward.

Regards

Peter

Added pull request for the D10 updates and updated the code to use service injections. Tested locally.

Tested locally with the patch and content check using `rendered_entity_full_view` input still works fine

The changes look good, could we merge it and tag a new release for D10 please?

Setting a min-width is probably not scalable because it only supports the default placeholder text. The overflow is caused by the min-width set for .chosen-choices, removing that should fix the overflow issue and it's up to users to set a min-width in their custom implementations.

Tested on our 9.5.11 site and it's working fine. Could you provide more information on this? Make sure you have updated the field widget to use Chosen. Otherwise, even though Chosen will turn your select list into a chosen widget by default, the hooks in chosen_order won't kick in.

locale-language_list-3119280-2.patch is conflicting with 9.5.9, rerolling a new patch

Hi @bgreco,

That would be the same reason as having Media translatable in general I think. There are many cases we would need to translate a media and have it identified as the same Media object. For example, an image with text, a youtube video or a PDF file that is available in different languages.

peterwcm → created an issue.

Thank you @damienmckenna for maintaining this. I saw that all child issues seem to be resolved, will there be more work before this is ready to be released? Just want to get a rough idea of when this will be released so we can make a decision on whether or not to apply some patches for the time being. Cheers.