Account created on 17 March 2016, over 8 years ago
  • Drupal Backend Developer at SeeD EM 
#

Merge Requests

More

Recent comments

🇵🇪Peru alyaj2a

Hello paraderojether! I already uploaded the changes! Thanks!

🇵🇪Peru alyaj2a

Thanks paraderojether! I am going to check it.

🇵🇪Peru alyaj2a

Hello avpaderno, I uploaded the corrections that I indicated in this issue appeared to me with PHPCS.

🇵🇪Peru alyaj2a

Hello, I uploaded all the corrections, except these.
In this case, I need change ta names ta functions because is not in lowerCamel format, a example: computeCKEditor5PluginSubsetConfiguration => computeCkeditor5PluginSubsetConfiguration

FILE: .../web/modules/custom/linkit/src/Plugin/CKEditor4To5Upgrade/Linkit.php
-------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
-------------------------------------------------------------------------------------------------
27 | ERROR | Public method name "Linkit::mapCKEditor4ToolbarButtonToCKEditor5ToolbarItem" is
| | not in lowerCamel format
34 | ERROR | Public method name "Linkit::mapCKEditor4SettingsToCKEditor5Configuration" is not
| | in lowerCamel format
59 | ERROR | Public method name "Linkit::computeCKEditor5PluginSubsetConfiguration" is not in
| | lowerCamel format
-------------------------------------------------------------------------------------------------

In this case there are thow name classes into condicionals, and the issue say: Class name doesn't match filename; expected "class UpgradePathCompletenessTest".

-------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------
14 | ERROR | Class name doesn't match filename; expected "class UpgradePathCompletenessTest"
-------------------------------------------------------------------------------------------------

Help me please, I don't know what to do with these remaining issues. Thanks.

🇵🇪Peru alyaj2a

Hello paraderojether, I tried testing on the branch 3472429-coding-standard-issues and also on 2.x, but I wasn't able to get the result you're showing. Could you please help me understand what I might be missing? Thank you!

🇵🇪Peru alyaj2a

Hello, I was going to correct the Issues that appear when passing phpcs, however in the branch itself these Issues do not appear, so I will close this. Thanks.

🇵🇪Peru alyaj2a

Hello. I believe the request made here might be addressing a very specific use case. I'm wondering if this should be developed within this module or perhaps considered for the Media module of Drupal core instead. It might be worth discussing whether the functionality should be part of the core's broader capabilities or remain in a more specialized module like this.

🇵🇪Peru alyaj2a

I think the administrator of the module page will need to write the information into the module's page. I'm going to move this issue to "Needs Work".

🇵🇪Peru alyaj2a

alyaj2a changed the visibility of the branch 3368623-list-alternative-modules to hidden.

🇵🇪Peru alyaj2a

Hello, we have translated the landing page into Spanish. I have also attached the document in XLS format. Here is the Google Docs link to the Spanish translation.
This is the document on Google Docs with the translation into Spanish.

Thank you!

🇵🇪Peru alyaj2a

In my case there was a space above <?php in settings.php. I removed this space y the problem resolved.

🇵🇪Peru alyaj2a

In my case there was a space above <?php in settings.php. I removed this space y the problem resolved.

🇵🇪Peru alyaj2a

Thanks! I test two patches, the number #27 🐛 Ajax Pager broken after upgrade 10.0.9 to 10.1.2 Needs work and #44 🐛 Ajax Pager broken after upgrade 10.0.9 to 10.1.2 Needs work , the two patches worked for me.

🇵🇪Peru alyaj2a

Thanks, this patch worked for me, Drupal 10.1.4 and the module in version 8.x-1.1.

🇵🇪Peru alyaj2a

Thank you. #30 🐛 Improve performance for range widgets Needs review . This patch worked for me.

🇵🇪Peru alyaj2a

Patch #273 🐛 Ajax support / Use behaviors Needs work worked perfectly, with this patch recaptcha in modals was solved. Thanks.

🇵🇪Peru alyaj2a

I used patch #9 📌 Automated Drupal 10 compatibility fixes Fixed and it installed fine, the detail is when you don't configure it, anonymously I get an error which I adjusted with a validation, I don't know if it's the right way. Help!.

Production build 0.71.5 2024