Ho Chi Minh City
Account created on 8 April 2015, almost 10 years ago
  • Drupal Developer at AthenaΒ 
  • Drupal Developer at FIDESIOΒ  …
#

Merge Requests

Recent comments

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City
πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I've updated the README.md file with the guideline for the How to configure part.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

phannphong β†’ made their first commit to this issue’s fork.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I see this issue is fixed with the patch file from this ticket #3827041 πŸ“Œ Automated Drupal 10 compatibility fixes Needs review

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

This is duplicated with the issue #3287041 πŸ“Œ Automated Drupal 10 compatibility fixes Needs review

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

You should use only one of this module or RobotsTxt module because both modules have the same path /robots.txt. If you have different domains and you want to have different "robots.txt" content, you can install this module and configure from the following path /admin/config/domain/robots_txt/{domain}.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

An update with fixing coding standards.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

phannphong β†’ changed the visibility of the branch 3287041-automated-drupal-10 to active.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

phannphong β†’ changed the visibility of the branch 3287041-automated-drupal-10 to hidden.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I'm pretty sure the "Trash" page (/admin/content/trash) will lead to 404 page because in the routing, the trash page doesn't have any Controller to handle this path.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

The patch #17 works like a charm. Thank you!

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

This patch works with the version 1.2.13 when the ticket πŸ› Ajax support / Use behaviors Fixed is fixed.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

Hi, you can check this issue here πŸ“Œ [PP-1] SEPA Support Postponed

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I've updated the patch to work with PHP 7.x

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

phannphong β†’ created an issue.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

On Drupal 10, we need this patch and the patch from the issue Google Places Autocomplete Not working Drupal 10 hook_field_widget_WIDGET_TYPE_form_alter is deprecated πŸ› Google Places Autocomplete Not working Drupal 10 hook_field_widget_WIDGET_TYPE_form_alter is deprecated RTBC to bring the modules works back.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

This patch is not enough to let module works because we have an error related to this issue Google Places Autocomplete not working $(...).once is not a function in Drupal10 πŸ› Google Places Autocomplete not working $(...).once is not a function in Drupal10 RTBC .
So, we need to apply both patches to bring the module works again.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

@greggles, yes, it works well.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

The path #18 is working well and I think it can be delivered asap.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I can confirm that the patch #2 is good. But the patch is fixed only for "Add" button without fixing overlap if we have many paragraphs (at least we have more than 1 line) in the modal. It's referenced in the issue CSS overlap in D10 Claro πŸ› CSS overlap in D10 Claro Fixed

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I can confirm that #17 is working as expected.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

Yes, I agree with you because the patch will break some other stuffs. I have to back with the theme based on Claro.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

Hi stBorchert, the patch πŸ› Add-in only works if active theme is claro or gin RTBC from MR 12, can work without Claro or Gin enables so in my project, I applied above patch for Paragraphs Features and apply this patch for Paragraphs Editor Enhancements.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I created a new patch in case the patch in the issue [PP-1] SEPA Support πŸ“Œ [PP-1] SEPA Support Postponed is applied with the version 8.x-1.1

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City
πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

The current patch works with rc6 but not for 8.x-1.1 version so I created a new patch here.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

For the modal layout is broken, I created an issue on paragraphs_ee https://www.drupal.org/project/paragraphs_ee/issues/3404844 πŸ› Modal layout is broken if the Claro theme is not installed/set as administration theme Active .

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I'm facing the same issue and tried some ways to fix it but not success. Does anyone knows how to fix it or sharing you experience is this case. Thanks.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I faced exactly the issue here and the patch #6 fixes the error on Drupal 10.1.6.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

menu_link_destination_form_menu_link_content_menu_link_content_form_alter is not called with Drupal 10 also. I agree with update by v1nk.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

As @aasarava mentioned, this issue occurs when using config_split with the field_ui set to enable only in the local/dev environment, as configured in the Complete Split section (Complete Split: Configuration listed here will be removed from the sync directory and saved in the split directory instead. Modules will be removed from core.extension when exporting (and added back when importing with the split enabled.)) so the field_ui.settings.yml configuration file is missing, deleted, or removed.

I believe we don't need to apply any patch to fix this issue because we only need to place the field_ui.settings.yml configuration file in the local/dev folder (folder name depends on what you configure in the Config Split module) or add it to git (Remember that we shouldn't place this configuration file in the sync folder because we only want to enable the field_ui in the local/dev environment).

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I have made updates based on patch #20 to create a new patch that can be applied to version 3.0.0-beta1.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

As I mentioned in the previous comment, when the patch #6 can fix the problem. However, this patch will trgger new warning message:

Warning: Undefined array key "#allowed_tags" in Drupal\webform\Plugin\WebformElementBase->prepare() (line 813 of modules/contrib/webform/src/Plugin/WebformElementBase.php).

Therefore, I've uploaded a new patch to address this new issue.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I'm facing the same problem and the patch at #6 can fix the problem.
I'm using webform version 6.2.0-beta6.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I think your requirement doesn't belong this module. I guess you're using the address module, if so, you can following this document https://docs.drupalcommerce.org/commerce2/developer-guide/customers/addr... to limit the available countries.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I don't face this issue when user logout but face the exactly issue when user login.
The patch MR!50 is solved my problem when user login.

πŸ‡»πŸ‡³Vietnam phannphong Ho Chi Minh City

I've updated the patch file to fix warning in the module file also.

Production build 0.71.5 2024