Hello @charlliequadros
I have checked the MR and it seems that the suggested changes have been implemented
Moving the issue to RTBC and let's wait for some more feedback from @alexpott or @longwave,
Hello @koustav_mondal
Ihave checked the MR note and it seems that the suggested changes have been implemented
Moving the issue to RTBC and lets wait for some more feedbacks
Hello Jonasanne
I have reviewed your MR and it seems that it is removing the dependency from composer.json
Moving the issue to RTBC and wait if others to review
Hello ankitv18
As per suggestions, I have rebased the branch with 2.x
Please review and suggest any other changes required
adwivedi008 → made their first commit to this issue’s fork.
Hello Shweta
I have tested the MR and it seems that it is resolving the issue
Moving the issue to RTBC
Hello @Doxigo,
I have added a few checks for preventing errors, and also added the exception handling for the errors provided
Please review and suggest any other changes required
Moving the issue to needs review
adwivedi008 → made their first commit to this issue’s fork.
The changes seem fine to me and are working as expected
However, I have tested the changes based on the MR, as the Steps to reproduce are missing
Moving the issue to RTBC, and also appreciate if we add steps to reproduce
Adding tag "Needs steps to reproduce" so that we have some more RTBC before moving forward with it
As the Status of the issue was RTBC before merge conflict, I guess @sivaji has only made a merge to resolve conflict
The MR seems fine to me
Moving its status back to RTBC and let's wait for some more RTBC, then move forward with the changes
Hello @adamps
I have implemented the suggestion provided for extending support to PHP 8.4.
Please review and suggest any other changes is required
Moving the issue to Needs Review
adwivedi008 → made their first commit to this issue’s fork.
Hello @ankitv18
I have tested the MR on D-11 and it is working as expected the module settings page is working fine
I also checked the MR and there are some comments added on MR by you if they are resolved then mark them as resolved
If the issue is for different Drupal versions then maintainers can create 2 different releases for different versions
Moving the issue to RTBC
Hello Undersound3 and Everyone
I was checking out for an edit link and that was neither looking good nor handly
That's why I thought it would be better if we use operational links here instead of an edit link
I have modified the MR accordingly
Please review and suggest any other changes required
I have added screenshots for both
adwivedi008 → made their first commit to this issue’s fork.
Hello @Joachim
Have updated the documentation according to the provided description
Please review and suggest any other changes required
adwivedi008 → made their first commit to this issue’s fork.
Hello @quietone
I have added the recommended changes
Please review and suggest any other changes required
adwivedi008 → made their first commit to this issue’s fork.
Fixed the issues reported by Drupal coding standards
Please review and let me know if any other changes are required
adwivedi008 → created an issue.
Hello @arturopanetta
Created the MR with the fix for the issue
Please review and let me know if any other changes are needed
Moving the issue to needs review
adwivedi008 → made their first commit to this issue’s fork.
Hi @xmacinfo
Created the MR with the proposed solution
Please review and let me know if any other changes are needed
adwivedi008 → changed the visibility of the branch 3507917-move-fakeobjects-logo to active.
adwivedi008 → changed the visibility of the branch 3507917-move-fakeobjects-logo to hidden.
adwivedi008 → changed the visibility of the branch 3507917-move-fakeobjects-logo to active.
adwivedi008 → changed the visibility of the branch 3507917-move-fakeobjects-logo to hidden.
adwivedi008 → made their first commit to this issue’s fork.
Hello @nod
Can you please suggest what other changes are needed on this
Or do we need to create another issue related to this for cleanup
Hello @daniel
I have taken a look at MR and it seems to be correct
Moving the issue at RTBC
Thank You let's wait for some more feedbacks
Hello @hondaman900
The module is originally developed for D8 or higher version
If you still want to modify your can create a feature request and raise a MR, would love to merge and release for D7 too
adwivedi008 → made their first commit to this issue’s fork.
@joachim I have updated the readme file with the comments and the settings
Please review and merge
adwivedi008 → made their first commit to this issue’s fork.
@smust grave I have updated the MR according to the suggested changes
Please review
adwivedi008 → made their first commit to this issue’s fork.
MR 61 seems fine to me
tested on vanilla Drupal and now buttons are translatable
Moving the issue to RTBC
@dhruv.mittal
Can you please check again, the MR against which version you are creating the MR I guess it should be created against 4.0.x and the branch should also be taken from 4.0.x
Moving this issue to Needs work as there are more than 200 commits in MR
Updated the dependency as per the issue description comments
Please review
adwivedi008 → made their first commit to this issue’s fork.
Reverted the out-of-scope changes.
please review
adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to active.
adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to hidden.
adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to active.
adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to hidden.
adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to hidden.
adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to active.
adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to hidden.
Is there any update on this issue? This is an issue if we are using Ajax and a better-exposed filter together
Any workaround would also be appreciated
Is there any update on this issue? This is an issue if we are using Ajax and a better-exposed filter together
Any workaround would also be appreciated
The issue has been fixed and resolved
Thanks @bhaveshdas and @prem
Moving the issue to fixed
Test the MR-8 changes with CKE-5 and Drupal-10 and its working fine as the feature is expected
Moving the issue to RTBC
Merged D-11 related changes
Hi everyone, Thanks for working on the issue
Can someone create a merge request with the latest changes
I am facing the same issue and patch #50 doesn't seem to resolve the issue for D-10.2.6
any suggestions
@rveillard Do you find any solution for this
I am also getting the same issue for video component
@jochim updated the MR as per your suggestions
Apology for the bulk changes as I get confused about bringing the whole doc changes vs only document part changes
Moving the issue to needs review.
@joachim
I have updated the documentation using
https://www.drupal.org/files/issues/2020-05-29/3040427-37-migrate_plus_m... →
and
https://www.drupal.org/files/issues/2020-05-17/3040427-32-migrate_plus_m... →
as reference as you have mentioned in #5
Closed MR of #7 as that was not correct and was opened by mistake apology for that
Please review the changes and inform if any other changes are required
adwivedi008 → made their first commit to this issue’s fork.
Hi everyone,
I want to work on this issue can someone guide me to which section can we add the module and remove from core
Created MR for all the issues raised over coding standard command
Checked using the following command.
vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,info,txt,md,yml ./
Moving the issue to Needs Review
adwivedi008 → made their first commit to this issue’s fork.
Verified the changes mentioned on MR-7857
The padding is 12px 16px over the menu items child
Attaching a screenshot as a reference
Moving the issue to RTBC
RTBC++
adwivedi008 → made their first commit to this issue’s fork.
@finnsky Can confirm the issue only happens when the inspect element is open and the toolbar gets a scrollbar on it
I am attaching a screen record as a reference and looking into this issue.
adwivedi008 → made their first commit to this issue’s fork.
@ckrine @m4olivei
Implemented the suggested changes but got Merge error
Can we resolve this by rebasing with 11.x
adwivedi008 → made their first commit to this issue’s fork.
Rebased '3443567-update-ckeditor-5-11.x' branch with 11.x
adwivedi008 → made their first commit to this issue’s fork.
adwivedi008 → made their first commit to this issue’s fork.
Reverted the remaining issue.
Setting the issue to Needs review
Reverted out-of-scope changes suggested by @smustgrave on #14
Setting the issue to Needs review
adwivedi008 → made their first commit to this issue’s fork.
Rebased the branch with 11.x
Please review if any other changes are required.
adwivedi008 → made their first commit to this issue’s fork.
Patch at #8 works great for 9.5.x and 10.1.x but it failing for D-10.2.x
@smustgrave I was not sure about the permissions that's why I haven't worked on it.
Thanks @spuky
Working fine on D-9.5
Any update on this for D-9.5 or D10
Worked on Pt. 1 mentioned in the proposed solution.
adwivedi008 → made their first commit to this issue’s fork.
Moving the issue back to Need Review as it was already in Needs review state
Updated the MR-5052
Rebased to 11.x
Please review if any other changes are required.
adwivedi008 → made their first commit to this issue’s fork.
I have added a new configuration key to the config form of the block to show and hide the count of searches
Attaching a screenshot for reference
Moving the issue to Needs Review.
adwivedi008 → made their first commit to this issue’s fork.
@adarshv Tested the MR-3 against D-10 but it still needs work as the header is still broken comparatively
So moving the issue to needs work
Attaching a screenshot for reference.
Tested the MR-12 and it fixes the issue
Attaching the screenshot as a reference.
Moving the issue to RTBC
adwivedi008 → made their first commit to this issue’s fork.
@wim fixed the phpcs issue and updated the MR PLease review
Moving the issue to needs review
adwivedi008 → changed the visibility of the branch 3440878-fix-return-typehinting to active.
adwivedi008 → changed the visibility of the branch 3440878-fix-return-typehinting to hidden.
Created MR with the proposed solution over LangcodeRequiredIfTranslatableValuesConstraintValidator::validate()
Please review
Moving the issue to Needs review.