Account created on 31 May 2016, over 8 years ago
#

Merge Requests

Recent comments

πŸ‡ΊπŸ‡¦Ukraine danmer

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

πŸ‡ΊπŸ‡¦Ukraine danmer

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

πŸ‡ΊπŸ‡¦Ukraine danmer

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

πŸ‡ΊπŸ‡¦Ukraine danmer

Hello @s_leu, I tried to apply the patch but faced an issue, so I slightly updated it. Can you please check this patch?

πŸ‡ΊπŸ‡¦Ukraine danmer

Stable releases for this project are covered by the security advisory policy.

πŸ‡ΊπŸ‡¦Ukraine danmer

The code has been merged into the new release. Thank you, @jurgenhaas!

πŸ‡ΊπŸ‡¦Ukraine danmer

The code has been merged into the new release. Thank you @jurgenhaas!

πŸ‡ΊπŸ‡¦Ukraine danmer

I have updated the related issue and am now waiting for a response.

πŸ‡ΊπŸ‡¦Ukraine danmer

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.

πŸ‡ΊπŸ‡¦Ukraine danmer

@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!

πŸ‡ΊπŸ‡¦Ukraine danmer

Thank you @jurgenhaas tomorrow I'll try to resolve this and other issues you addressed

πŸ‡ΊπŸ‡¦Ukraine danmer

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.

πŸ‡ΊπŸ‡¦Ukraine danmer

The fix is ​​added in the new release 1.3.1.

πŸ‡ΊπŸ‡¦Ukraine danmer

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?

πŸ‡ΊπŸ‡¦Ukraine danmer

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.

πŸ‡ΊπŸ‡¦Ukraine danmer

Tested, and it looks good, and I added a small tweak to the patch to have a clear phpcs report

πŸ‡ΊπŸ‡¦Ukraine danmer

It looks good to be merged

πŸ‡ΊπŸ‡¦Ukraine danmer

Hello @vishalkadam, thanks for the review. I have updated the repo: https://git.drupalcode.org/project/zoom_video with all fixes for the phpcs

πŸ‡ΊπŸ‡¦Ukraine danmer

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.

πŸ‡ΊπŸ‡¦Ukraine danmer

Fixed in the lates release

Production build 0.71.5 2024