🇧🇪
Account created on 2 August 2012, over 12 years ago
  • Technical Lead at iO 
#

Merge Requests

More

Recent comments

🇧🇪Belgium JeroenT 🇧🇪

The new release is working fine here, Thanks!

🇧🇪Belgium JeroenT 🇧🇪

jeroent made their first commit to this issue’s fork.

🇧🇪Belgium JeroenT 🇧🇪

I made a mistake in my previous comment. @dqd, you are the owner of the MR, can you edit the target branch to 5.0.x?

🇧🇪Belgium JeroenT 🇧🇪

#3484837-10: Create D11 compatible version seems unrelated to the Drupal 11 update and can be handled in a separate issue.

Also, I don't think it's necessary to drop Drupal 10 support, so I would also keep that version in the core_version_requirement property and in composer.json.

🇧🇪Belgium JeroenT 🇧🇪

@heddn,
I can't find any feedback on the MR?

🇧🇪Belgium JeroenT 🇧🇪

2 weeks have passed and @markdorison, one of the co-maintainers, also agrees.

🇧🇪Belgium JeroenT 🇧🇪

Closing this as a duplicate of 📌 Fix the issues reported by phpcs Active

🇧🇪Belgium JeroenT 🇧🇪

The patch no longer works since 💬 Custom entity types Active was committed.

🇧🇪Belgium JeroenT 🇧🇪

I also closed Adds support for translatable fields Closed: duplicate as a duplicate since that is also fixed here. Because of the changes in .routing.yml file, the entity is loaded immediately in the right language, so translated file fields are already handled.

🇧🇪Belgium JeroenT 🇧🇪

Closing this as a duplicate of 💬 Custom entity types Active since that is also fixed over there.

🇧🇪Belgium JeroenT 🇧🇪

This is a nice improvement for this module.

I took the patch in #5, created a merge request of it and made a couple of changes.

I also closed 🐛 Check if node is valid Closed: duplicate as a duplicate since we no longer need to check if the entity is a valid entity because this now happens automatically because of this changes in the routing.yml file:

  options:
    parameters:
      entity:
        type: entity:{entity_type}
🇧🇪Belgium JeroenT 🇧🇪

I'm closing this as a duplicate of 💬 Custom entity types Active .

Once that issue is fixed, this check is no longer necessary because of the changes in the routing.yml file.

🇧🇪Belgium JeroenT 🇧🇪

jeroent changed the visibility of the branch 3197043-custom-entity-types to hidden.

🇧🇪Belgium JeroenT 🇧🇪

jeroent changed the visibility of the branch 3197043-custom-entity-types to active.

🇧🇪Belgium JeroenT 🇧🇪

📌 Automated Drupal 11 compatibility fixes for download_all_files Needs review will drop support for Drupal 8 and 9 so I guess we can close this one.

🇧🇪Belgium JeroenT 🇧🇪

jeroent made their first commit to this issue’s fork.

🇧🇪Belgium JeroenT 🇧🇪

Tests are green again.

🇧🇪Belgium JeroenT 🇧🇪

jeroent made their first commit to this issue’s fork.

🇧🇪Belgium JeroenT 🇧🇪

I contacted Dennis Cohn via the contact form on drupal.org about this issue.

🇧🇪Belgium JeroenT 🇧🇪

jeroent made their first commit to this issue’s fork.

🇧🇪Belgium JeroenT 🇧🇪

Currently, it's not possible to translate the templates because the configuration is not translatable.

🇧🇪Belgium JeroenT 🇧🇪

I added some changes to MR1 that adds D11 compatibility and fixes all validations tasks in the Gitlab pipeline.

🇧🇪Belgium JeroenT 🇧🇪

jeroent made their first commit to this issue’s fork.

🇧🇪Belgium JeroenT 🇧🇪

I just created a new release (1.0.0-alpha3) which fixes a couple of bugs.
I wasn't able to reproduce this error. Does this problem still occur in that version for you?

🇧🇪Belgium JeroenT 🇧🇪

Sorry for the late response...

I started working a bit on this module and first wanted to write some basic test covarage, which I did in #3488487: Add test coverage .

I was able to reproduce this error and included this patch in that issue. Thanks for the work.

🇧🇪Belgium JeroenT 🇧🇪

How is this a problem? The only reference to ProfileInterface is in a php comment, which is fully namespaced. So an import is not necessary?

🇧🇪Belgium JeroenT 🇧🇪

I made the changes suggested by the maintainer to the MR.

@naveenvalecha, since you created the MR, can you edit the target branch to 5.0.x?

🇧🇪Belgium JeroenT 🇧🇪

jeroent made their first commit to this issue’s fork.

🇧🇪Belgium JeroenT 🇧🇪

I tried to reproduce this error.
I also wrote a test, but it seems that this is working:
https://git.drupalcode.org/project/field_fallback/-/jobs/3366749

Am I missing something?

🇧🇪Belgium JeroenT 🇧🇪

jeroent made their first commit to this issue’s fork.

🇧🇪Belgium JeroenT 🇧🇪

Merged to 2.0.x. Thanks!

🇧🇪Belgium JeroenT 🇧🇪

jeroent made their first commit to this issue’s fork.

Production build 0.71.5 2024