Balneário Camboriú
Account created on 30 November 2017, over 7 years ago
#

Merge Requests

More

Recent comments

🇧🇷Brazil isa.bel Balneário Camboriú

Tested and it's working smoothly.
Check the screenshot for evidences.

🇧🇷Brazil isa.bel Balneário Camboriú

Tested and the export is working smoothly.
Check the screenshot provided.

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel made their first commit to this issue’s fork.

🇧🇷Brazil isa.bel Balneário Camboriú

Ready to be reviewed

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel created an issue.

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel made their first commit to this issue’s fork.

🇧🇷Brazil isa.bel Balneário Camboriú

I checked the PR and all the checks passed, therefore I'm changing this to needs review again.

🇧🇷Brazil isa.bel Balneário Camboriú

I found what's wrong, closing this issue

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel created an issue.

🇧🇷Brazil isa.bel Balneário Camboriú

MR 1 hidden, but I can't close it

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel changed the visibility of the branch 3453692-hookentityreferenceselectionalter-is-undocumented to hidden.

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel changed the visibility of the branch document-hook_entity_reference_selection_alter to active.

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel changed the visibility of the branch document-hook_entity_reference_selection_alter to hidden.

🇧🇷Brazil isa.bel Balneário Camboriú

I created another Merge Request to run phpcs against the latest version of the module.
All adjustments have been made.
Ready to be reviewed

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel made their first commit to this issue’s fork.

🇧🇷Brazil isa.bel Balneário Camboriú

I rebased the current branch using the latest version of core. Ready to be reviewed

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel made their first commit to this issue’s fork.

🇧🇷Brazil isa.bel Balneário Camboriú

Please add steps to replicate the issue, because with a vanilla Drupal 11.1 and trying to install this module, no errors were thrown. See attached screenshot.

🇧🇷Brazil isa.bel Balneário Camboriú

Documentation added, moving to Needs Review status

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel made their first commit to this issue’s fork.

🇧🇷Brazil isa.bel Balneário Camboriú

The documentation seems complete and correct now, I will set this to Needs Review

🇧🇷Brazil isa.bel Balneário Camboriú

Thank you for your suggestion @codebymikey, I added a protected method specific for this, refactored the code to be more comprehensive, and also added this part to the code, without the JSON check, as it's not needed in this case, and what is also suggested in the WP git issue here.

🇧🇷Brazil isa.bel Balneário Camboriú

I had some issues after updating to the latest stable version of the module, but using the patch from #84 worked for me!

🇧🇷Brazil isa.bel Balneário Camboriú

Hi @MegaChriz, thank you for your reply, we were already using that patch, but from comment #56 so it was outdated, I already updated it and applied it, and now the code is already in place. Thanks again!

🇧🇷Brazil isa.bel Balneário Camboriú

Hello, after updating to the latest version of this module (beta4), which includes this MR originated from this issue I began to have this error:

ArgumentCountError: Too few arguments to function Drupal\feeds\Feeds\Target\File::__construct(), 9 passed in web/modules/contrib/feeds/src/Plugin/Type/Target/MediaTargetBase.php on line 49 and exactly 11 expected in Drupal\feeds\Feeds\Target\File->__construct() (line 101 of modules/contrib/feeds/src/Feeds/Target/File.php).

Therefore I created a patch specifically for this, if you guys can review it, I really appreciate it.

🇧🇷Brazil isa.bel Balneário Camboriú

Based on the issue opened on Gutenberg repository: https://github.com/WordPress/gutenberg/issues/41124

And solution provided here: https://github.com/WordPress/wordpress-develop/blob/6.0/src/wp-includes/...

I have created an MR to add the changes necessary to recreate the group inner container. Which fixed the issue for me.

🇧🇷Brazil isa.bel Balneário Camboriú

Patch #3 works for me.

🇧🇷Brazil isa.bel Balneário Camboriú

We have created a patch for this issue. I will attach it here to be reviewed.

🇧🇷Brazil isa.bel Balneário Camboriú

isa.bel created an issue.

Production build 0.71.5 2024