Grotteria (RC)
Account created on 8 August 2011, almost 13 years ago
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

@flavio_russo

  1. Install docker https://docs.docker.com/engine/install/ubuntu/
  2. Install DDEV https://ddev.readthedocs.io/en/stable/users/install/ddev-installation/#l...
  3. run $ bash <(curl -s -H "Cache-Control: no-cache" "https://git.drupalcode.org/project/bootstrap_italia/-/raw/2.8.x/var/bin/build-ddev-installation.sh")
๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Hi Flavio,
if you install ddev (it helps you create a development environment with the right requirements, yours does not have the right requirements), by launching the command described here https://git.drupalcode.org/project/bootstrap_italia/-/tree/2.x#how-to-st... it will do all the work described in the guide automatically for you.

---

Ciao Flavio,
se installi ddev (ti aiuta a creare un ambiente di sviluppo con i requisiti giusti, il tuo non ha i requisiti giusti), lanciando il comando descritto qui https://git.drupalcode.org/project/bootstrap_italia/-/tree/2.x#how-to-st... farร  tutto il lavoro descritto nella guida automaticamente per te.

Ps. Flavio, su drupal.org si usa l'inglese (anche male o malissimo, ma si usa l'inglese), รจ una regola della community.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

ok @aaron.ferris, mark fixed, thanks :)

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

The theme has a workaround for using icons in this case. Go to subtheme settings -> "Libraries" -> "CDN Settings" -> "Icons from CDN"; set the path until you reach sprites.svg.

Obviously remember not to bring this setting into production if it is not needed.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)
๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

I updated the dependencies and upgraded to videojs 8.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

I updated the dependencies and upgraded to videojs 8.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

I am closing this page as php_ffmpeg has been updated to drupal 10

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

@starglide, @Hritick is right, on drupal.org they speak in English.

@alice, however you are right it's a bug. A bug with no negative effects though.

The bug is caused by this line of code: https://git.drupalcode.org/project/bootstrap_italia/-/blob/2.x/modules/b...

Drupal automatically creates a single twig namespace that points to the "templates" folder of each module or theme, for example the "@foo" twig namespace will point to the "foo/templates/" folder of the "foo" module.

In the specific case for the "bootstrap_italia_paragraph" module, drupal automatically builds a namespace "@bootstrap_italia_paragraph" which points to the "/themes/contrib/boostrap_italia/modules/bootstrap_italia_paragraph/templates" folder. But at the same time a new namespace is defined on line 16 of "bootstrap_italia_paragraph.info.yml" which overwrites the one automatically created by drupal.

Obviously my intentions when I wrote this piece of code were not to override the automatic namespace that drupal creates, so it's a bug.

I don't think the proposed solution is correct as the namespace is used in many other parts, so making the proposed change would break other parts of code.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

@luca_cracco thanks for your contribution, sorry if I don't apply your patch but two lines have already been fixed in a similar way:

If you want to produce a new patch that fixes the remaining lines with the same method I will apply them.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Hi, thanks for the report. I couldn't reproduce the problem, could you provide more details?
If you can explain all the steps to reproduce the problem it is easier to fix.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Ciao @McAle, ottima osservazione! enable-all-tooltips e load-fonts non caricano automaticamente le dipendenze.

Penso che la risoluzione del problema sia un po' diversa, cosidera che bootstrap viene fornito dal sub-theme ed esiste solo dopo che il sub-theme ha caricato le sue librerie, il caricamento avviene qua, quindi aggiungere come dipendenza bootstrap_italia/base (che non fornisce bootstrap) non risolverebbe il problema, quindi forse sarebbe opportuno spostare questi due js nel sub-theme in modo da far caricare le dipendenze in automatico?

Bisogna ragionarci un po', tu che ne pensi?

Comunque l'errore che ti viene segnalato che bootstrap non esiste sicuramente va ricercato nel caricamento della libraries del tuo sub-theme.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

@sidgrafix Thank you and what a wonderful thing you are doing.

For the queue module I opened a specific issue https://www.drupal.org/project/streaming/issues/3393523 โœจ Feature: streaming_transcode_queue Active , click on the button to create your fork on git and then send a merge request.

For other issues I think it is better to open other issues so we can keep track of everything.

Thanks for what you're doing

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

arturopanetta โ†’ created an issue.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Thanks for the feedback, I'm carrying out this module in my free time (which isn't much). If you want to send some code it's really welcome <3

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)
๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Hi sidgrafix,
this is first commit for D10 compiance https://git.drupalcode.org/project/streaming/-/commit/83525409b950317912...

See also https://www.drupal.org/project/php_ffmpeg/issues/3365929 โœจ Port to D10? Closed: duplicate

Ciao :)

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

I sent a merge request to make the form D10 compliant. The code sent also contains patch #9 of #3256578, I had to insert patch #3256578 #9 as the code I sent would not work on Drupal 10.
The rest should go smoothly.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

arturopanetta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Hi, I can't reproduce the error, what version of node are you using? Currently version 18 of nodejs is compatible.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)
๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Ciao @kopeboy,
I wasn't able to reproduce the error.

AssertionError: $attributes, if set, must be an array or object of type \Drupal\Core\Template\Attribute in assert() (line 245 of core/lib/Drupal/Core/Template/TwigExtension.php)

Could you provide me with more information?

  1. Yes, you need to install drupal/components:^3.0@beta if you have Drupal 10. You can find installation instructions at https://git.drupalcode.org/project/bootstrap_italia/-/blob/2.3.x/README..... Since you are using 'DDEV', you might consider using the automated installation script as well: https://git.drupalcode.org/project/bootstrap_italia/-/blob/2.3.x/README.....
  2. Yes, you are right. This is one of the challenges we plan to address in the upcoming releases. If you'd like to help, please join us on our Telegram channel.
  3. This issue stems from a dependency on svg-sprite-loader, which we need for compiling SVG icons. Currently, there are no alternatives in consideration for replacing the library. I will bring up this topic in the next developer meeting. If you would like to participate, please join us on the Telegram channel as well.

Grazie

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

I tested patch #2 and it works, I can redirect logs to stdout container. Patch #3 which I was unable to apply as it tries to remove some lines of code related to module versioning.

I used to:
- Drupal 10.0.9
- log_stdout branch 1.x
- patches #2

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Thanks @AndersTwo <3

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

#18 @allexim No, when uninstalling the module it doesn't delete user-created content, it's a deliberate choice (deleting content didn't seem like a good choice).

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

#19 Hi @allexim about the 503 error I mentioned it in https://www.drupal.org/project/streaming/issues/3305459#comment-14671382 โ†’ . As for the load you can choose the number of Threads to use from the PHP FFMpeg settings

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Thanks @Bhanu951 for the patch, I'll release the updated version as soon as I can.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Thanks @apaderno i removed the files with comments only

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Thanks to vishal.kadam, apaderno and Rashmisoni for advice and help.

1. Fix PHPCS issues: bbdd40b9

2. Remove LICENSE file: eef9064d

3. Place css and js folders in the root directory instead of the src directory: c55cf3d2

4. phpstan issues: 7e9f53a2

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

I tested the patch with Drupal 10 and php 8.1 and it works fine.

๐Ÿ‡ฎ๐Ÿ‡นItaly arturopanetta Grotteria (RC)

Version 2 solves this problem with a workaround. Go to theme settings -> Libraries -> CDN Settings and fill the "Icons from CDN" field with the correct path.

Production build 0.69.0 2024