Bij make it fly geloven we sterk in Drupal en Open Source software in het algemeen. We maken dagelijks gebruik van vele open source componenten en de vele Drupal modules die beschikbaar zijn. We dragen actief ons steentje bij aan de community door zelf patches, modules en documentatie te voorzien, daarnaast zijn enkele teamleden ook actief in de Drupal User Group vzw.
Een overzicht van onze bijdragen aan de Drupal community vind je terug op onze Drupal.org pagina. Naast het actief meerwerken aan de Drupal community, zetten we ook andere ontwikkelaars op weg door vragen te beantwoorden op Drupal Answers.
I've setup this module and it works great!
I have a few content blocks which use the field type double_field (https://www.drupal.org/project/double_field) for example for the FAQ (Question / Answer). I notice all content from the double_field type is not translated when using the AI translation.
Setup a content block with a double_field and fill it with some content Run AI translate
f0ns created an issue.
Latest commits in 2.x have failing PHPCS jobs, so that still needs fixing.
Merge request has been updated, since the blocking issue is now resolved. PHPCS pipeline is passing now.
svendecabooter made their first commit to this issue’s fork.
I'm running MAMP on Mac OS X and I found this issue which helped me.
I'm adding mu solution here for future reference (if needed):
I was unable to send mails locally (and catch them with mailhog).
I got the following error:
An attempt to send an e-mail message failed, and the following error message was returned : Error creating transports: Unsupported sendmail command flags "/Applications/MAMP/Library/bin/mhsendmail --smtp-addr=0.0.0.0:1025"; must be one of "-bs" or "-t" but can include additional flags.
The solution was to add the following line in the php.ini file of the active php version:
; For Unix only. You may supply arguments as well (default: "sendmail -t -i").
; http://php.net/sendmail-path
sendmail_path = "/Applications/MAMP/Library/bin/mhsendmail -t --smtp-addr=0.0.0.0:1025";Thank you for the new release 🙏
I've tested the MR and everything works fine.
When trying to change an existing media item, I get the error message "Non-reusable blocks must set an access dependency for access control."
This is for the drupal.be website - so might be related to i18n, although we have only set up 1 language so far.
This can be worked around by removing said block, and creating a new one with the new image, but that's a bit cumbersome.
- Edit an existing page, e.g. Homepage. - Try to change the media item used, by providing a new one.
svendecabooter created an issue.
For the drupal.be website I set up a homepage banner block with an image to the left. The image contains a rather dark background.
When checking the site on mobile, the hamburger menu was not visible, because it consists of black lines. I have resolved this by choosing another image for the homepage banner, but it took me a few week until I noticed. So preferably we can provide a fix for this, so that the hamburger menu is still visible with a dark background image, or alternatively, provide a warning to content editors to make sure they do not use such an image, as to not break the experience for mobile users.
- Edit homepage - Edit "Homepage banner" block and upload an image with a dark background instead.
svendecabooter created an issue.
bramdriesen credited svendecabooter.
svendecabooter created an issue. See original summary.
Use return types where relevant, etc...
svendecabooter created an issue. See original summary.
Thanks for the MR. Manually updated the fix for the new OOP hook implementations
Use OOP hooks, as per [#3442349]
svendecabooter created an issue. See original summary.
Thanks - committed now.
This looks like a duplicate of #3347330: Modal Does not Close Automatically (possibly only on multilingual sites), caused by the absolute URL. Can you report back in that ticket if the problem still persists on the latest version of this module?
"Onze teamleden bouwen zelf ook mee aan ons geliefde Drupal, en daar zijn we trots op"