Saint-Denis (Réunion)
Account created on 29 March 2018, about 7 years ago
  • Responsable pôle Back-End & Lead Developpeur at Ecedi 
#

Merge Requests

More

Recent comments

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Sorry, I was on version 1.3 and this has been corrected on the dev version.
I close the issue

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

MR works fine ! Thanks.
I no longer see alerts.
Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

MR works fine !
I no longer see alerts.
Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)
🇷🇪Réunion Martygraphie Saint-Denis (Réunion)
🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Review and merged thanks !

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Merged thanks !

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Merged thanks !

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Thanks merged !

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Merged thanks

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)
🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Merged, thanks !

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hi @doxigo,
There's still the stylelint job, which I think may have more or less impact. Would you look into this? (I'm not the most knowledgeable on this subject)
I recommend that you test the update following the .module update (normally no impact).

Source : https://git.drupalcode.org/issue/gin_gutenberg-3509129/-/jobs/4500720

Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hi @jhuhta
I fixed cspell and added the dependencies (drupal/ai and drupal/key) in composer.json this will strengthen version management and allow PHPStan to perform the audit as a whole)
I let you test and tell me if ok for you
Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hello,
Yes ok, I'll create a new issue

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hi @doxigo,
Sorry, I forgot to draft the MR, the pipeline passes but there is still warning phpcs, cspell, stylelint
Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hi @just_like_good_vibes
I may have misspoken, I was talking about the table in the Drupal page and not the README. I'd like to update the page but I don't have access to edit it, which is reserved for the module maintainer.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hello @chandansha
Could you modify your MR to target the 2.0.x branch which is the one used by the module?
Thanks
Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie created an issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hi,
I creating a merge request to make tracking easier. I made the necessary modifications and also replaced the default pictogram with a rounded version, which will be more consistent with the other icons in the project (link to the free pictogram: https://www.iconpacks.net/free-icon/bluesky-blue-round-circle-logo-24461...). I placed Bluesky below Twitter in the admin form for better coherence (as suggested by @s3b0un3t).

Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

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

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hi @ktpm,
thank you for this issue! I downloaded the module yesterday and set it up with Gemini, I'm having the same problem, the prompt entered in the provider is not supported!

Your patch works very well for me!

Thanks

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hi @leontin,
I merged your patch and credited you directly in this issue : https://www.drupal.org/project/geomap_field/issues/3430747 📌 Automated Drupal 11 compatibility fixes for geomap_field Needs review
Thanks !

Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)
🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hi @tim-diels
Will you be able to check the patch?
Thank you in advance
Marc

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie changed the visibility of the branch 3449679-add-attributes-specifics-routes-to-link to active.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

martygraphie changed the visibility of the branch 3449679-add-attributes-specifics-routes-to-link to hidden.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

I confirm that patch #3446896 correctly displays the number of results. I close this issue.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

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

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

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

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

Hello,
I'm also reproducing the problem in version 3.x.
If I understand correctly the problem / solution proposed by @veronicaSeveryn
I proposed a fix following feedback from @veronicaSeveryn. Which only adds Domain entity URL generator when creating the variant. I'm not aware of the impact on other features.

🇷🇪Réunion Martygraphie Saint-Denis (Réunion)

I've created a new issue #3449679 , I'm closing this one again

Production build 0.71.5 2024