Drupal 10 upgrade

MR looks very sensible.

Tested with Drupal 10. Looks good.

+1

Upgrading to Drupal 10 works but when i send a test email with an attachment i get this error
"TypeError: is_file(): Argument #1 ($filename) must be of type string, bool given in is_file() (line 139 of ....../contrib/mailchimp_transactional/src/Plugin/Mail/Mail.php).
Any ideas?

Works for me too, be good to get a release out :)

Looks like this is working solution, is there a plan to release a D10 update?

Last outstanding module blocking my D10 migration.

any possibility of this getting merged?

+1 on RTBC. Tests are passing now as well.

Converting into the patch.

I noticed mailchimp_transactional doesn't support D8 but its submodules do. That doesn't make sense, since the submodules depend on the main module. I'll change that.

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

@DieterHolvoet what is the status of a new tagged release for D10?

The only thing I'd like to have committed before cutting a new release is  🐛 TypeError: Drupal\mailchimp_transactional\API::send(): Return value must be of type array, ConnectException returned Needs review . It still needs a thorough review.