Account created on 18 July 2018, almost 6 years ago
  • Backend Developer at iOΒ  …
#

Merge Requests

Recent comments

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

You can expect this in the near future! Will be working to get this up and running in the summer.

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

Alright will make a test case showing the issue and will make a branch going towards 11.x

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

@cilefen, i have altered the commit and made sure we can run al the tests.

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

there seems to be a fix within the community by combining the following: (this is for drupal 10.1.*)

-> https://www.drupal.org/project/drupal/issues/2910353#comment-12422438 πŸ› Prevent saving config entities when configuration overrides are applied Needs work Thread of information about the core issue
-> https://git.drupalcode.org/project/drupal/-/merge_requests/6268.diff (diff i created the core patch from)

If you then patch webform with the attached patch after also patching Drupal core then the Config entities cant be overriden anymore.

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

Thank you for the response guys! Appreciate it, will make a MR / PR this weekend with the changes needed. Ill also look into automated tests.

With friendly greetings

Remco

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

Dear Maintainer Team, I proposed a solution in https://www.drupal.org/project/shopify/issues/3414025 🌱 New version of the module Active

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

Dear Maintainer Team, I proposed a solution in https://www.drupal.org/project/shopify/issues/3414025 🌱 New version of the module Active

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

I do notice that Karl Shea (20 december) already removed the if statement but here is a patch for people who want to run latest version of gin and not have this problem.

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

Remco Hoeneveld β†’ made their first commit to this issue’s fork.

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

Remco Hoeneveld β†’ made their first commit to this issue’s fork.

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

@saschaeggi great! Thank you for the fast reply! Great to hear its fixed and you also made it a lot nicer code wise! Great stuff

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

Thanks for testing @kevinfunk!

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

Here is a fix where i actually fix the chevron that is still wrong in the latest dev.

πŸ‡³πŸ‡±Netherlands Remco Hoeneveld

I was not, now i am using the latest version, ill see still some issues, ill make a new patch based on the latest dev version!

Production build 0.69.0 2024