🇮🇳India @atul ghate

Hi @isholgueras,

Thanks for merging the MR.

I noticed that I wasn't credited for resolving this issue. Could you let me know if there's something I might have missed?

atul_ghate → made their first commit to this issue’s fork.

Hi @bernardopaulino i have resolved conflict please review.

atul_ghate → made their first commit to this issue’s fork.

atul_ghate → made their first commit to this issue’s fork.

I have reviewed and applied patch #6, and it resolved the issue for the multi-value field in Drupal 9.5.0. The patch provided a button to remove items when the field is set to multi-valued, as described in comment #6. See the attached image.
It can be moved to RTBC.

I have fixed remaining phpcs issue. please review and merge the MR

atul_ghate → made their first commit to this issue’s fork.

I have tested the MR, it resolved all phpcs issues, moving it to RTBC

HI @trackleft2 , I have worked on the suggested changes, please review.

atul_ghate → made their first commit to this issue’s fork.

Added a merge request (MR) to fix PHPCS issues, but one still remains. See the image below.

atul_ghate → made their first commit to this issue’s fork.

atul_ghate → made their first commit to this issue’s fork.

Added MR to fixed all phpcs issue please review and merge.

atul_ghate → made their first commit to this issue’s fork.

Added MR to fix all phpcs issue please.

atul_ghate → made their first commit to this issue’s fork.

atul_ghate → made their first commit to this issue’s fork.

I have unassigned myself as @mstrelan has provided the merge request

atul_ghate → made their first commit to this issue’s fork.

I have fixed the issue as per drupal coding standard, please review.

atul_ghate → made their first commit to this issue’s fork.

atul_ghate → made their first commit to this issue’s fork.

Hello Tim-diels,

I have tested with an existing zone ID, and it is not deleting the existing zone ID but placing it at the bottom of the cloudflare_stream.settings.yml file. When we remove the zone ID field and schema.

I will provide the MR.

Hi @tim-diels and apaderno, Thank you for the clarification, But I encountered phpcs issue in this MR https://git.drupalcode.org/issue/cloudflare_stream-3446508/-/jobs/1613819 with this version 3.0.x

I will privide MR to fix this phpcs issue.

atul_ghate → created an issue.

Please review the MR.

I have added the MR please review and merge.

I will try to fix the phpcs issue.

Hi @biguzis,
I tried to reproduce this issue using below steps.

1. clone the 3.0.6 branch
2. enable module and create one article
3. configure the Personal Access Restriction settings for this node i have created
4. Add Personal Access Restriction settings for Anonymous user and select How to show the page? this option as Access Denied (403) and
   Page not found (404) as well but i did not faced issue you have reported. i can see page is restricated for Anonymous user

please let me know if anything i have missed to reproduce this issue

I will work on this issue.

I will work on this issue

Hi @tim-diels, thank you for sharing this information and for the guidance on using Merge Requests for contributions. I'll make sure to follow the guidelines provided and transition to using Merge Requests in future contributions. I apologize for the PHPCS issues in my patch and will address them in future contributions.

Please review and RTBC below patch.

fixed the issue reported by phpcs please review the attached patch.

please review updated patch.

okay will update updated patch.

please review the patch.

I will work on this issue.

I will work on this issue.

Replaced core/jquery.once with core/once please review and merge.

I do not have Noopener Filter module on my drupal instance 9.5.11 but still am getting same issue while saving the text format ckeditor 5

I have attached a patch please review it.

I will work on this issue.

This issue has not been found in version 8.x-2.1
Here are the steps I took to reproduce the issue
1. clone the module with 8.x-2.1
2. installed and added a menu link
Drupal 9.2.7 and installed theme: Olivero 9.2.7

I will review this patch.

I have provided a patch to fix this issue please review.

I will work on thi issue.

I have reviewed and applied this MR it resolved the mentioned issue, moving to RTBC

I will review this patch.

Please review the patch.

I have reviewed and applied #3 MR but still, the checkbox is not correctly aligned will provide the updated patch.

Hello,
I will review this MR

I have tested this patch it resolved the mentioned issue, moving to RTBC

I will review this pach.

I will review this patch.

I will work on this.

please review the attached patch.

I will work on this issue.

I have resolved this issue and provided a patch please review.

I will work on this issue.

atul ghate → created an issue.

I have resolved this issue and provided a patch please review

I will work on this issue.

atul ghate → created an issue.

I have reviewed and applied this MR, looks good to me moving to RTBC.

I will review this MR

The Issue has been fixed by this MR, moving to RTBC.

I will review this MR