Hello here, I have forked the issue and updated the patch.
I have removed the changes ForumLeafConstraint
since the Forum module isnβt in core anymore.
Also, I have updated FileImageDimensionsConstraint
and FileNameLengthConstraint
danmer β made their first commit to this issueβs fork.
Hello there, I have tested the issue according to the steps to reproduce.
My setup:
Drupal Version: 11.2-dev
PHP Version: 8.3.14
Database Version: 10.11.10-MariaDB-ubu2204-log
The patch fixes the issue; see attached screenshots
Hello @s_leu, I tried to apply the patch but faced an issue, so I slightly updated it. Can you please check this patch?
Stable releases for this project are covered by the security advisory policy.
The code has been merged into the new release. Thank you, @jurgenhaas!
The code has been merged into the new release. Thank you @jurgenhaas!
I have updated the related issue and am now waiting for a response.
Hello there, I have added a new release:
https://www.drupal.org/project/zoom_video/releases/1.3.3 β
.
Let me know if there are still bugs.
@jurgenhaas I have faced an issue merging it via UI, so I merged it manually and will add it in the next release. Thank you!
Thank you @jurgenhaas tomorrow I'll try to resolve this and other issues you addressed
The patch:
danmer β created an issue.
Hello there, I had no time to finish this task before, but now I have added new release:
https://www.drupal.org/project/zoom_video/releases/1.3.2 β
and the repo: https://git.drupalcode.org/project/zoom_video
Let me know if I can do anything else.
The fix is ββadded in the new release 1.3.1.
The fix was added.
danmer β created an issue.
Hello @socialnicheguru. What kind of cooperation do you mean? Do you want to merge the two modules, or add them as part of the same parent category or something else?
danmer β created an issue.
Hello, @vishalkadam, @apaderno. I have added a new release of the module; please check and let me know what I can fix or add if needed.
Oops, wrong interdiff
Tested, and it looks good, and I added a small tweak to the patch to have a clear phpcs report
It looks good to be merged
Hello @vishalkadam, thanks for the review. I have updated the repo: https://git.drupalcode.org/project/zoom_video with all fixes for the phpcs
Hello @apaderno, Thank you for your reply. Iβve updated the branching in the project repo: https://git.drupalcode.org/project/zoom_video
Tell me if I can do anything else.
danmer β created an issue.
Fixed in the lates release
Fixed in the new release
danmer β created an issue.
danmer β created an issue.
Here is the patch
danmer β created an issue.