🇮🇳India @nidhi27

Hii,

I have created MR !5 to resolve phpcs issues.
Thanks!

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

Hii,

I have resolved the comments.

Thanks!

Hii,

I have created MR !10 for this feature request. Let me know if any changes are needed.
Thanks!

Hii,

I have tried to resolve some of the errors. But there are still many errors that needs to be taken care.
Thanks!

nidhi27 → changed the visibility of the branch 3448212-deleting-a-content to hidden.

Hii,

It will be good to work on this after merging the MR of https://www.drupal.org/project/content_model_documentation/issues/3517038 📌 Fix PHP CS errors Active to avoid any conflicts. Let me know your view. Thanks!

Hii,

I have resolved all the phpcs errors.
Thanks!

Hii,

I have removed drupal_flush_all_caches(); from both the files.
Thanks!

Hii,

I have added rebuild and now after saving the entity it is not throwing any error. Let me know if this is the correct way or not.
Thanks!

Hii,

I have created the test files in https://www.drupal.org/project/front/issues/3438841 ✨ Make "Disable front page redirects for the administrator role" checkbox dependent using Form States API Active . It would be good to add tests for this after merge to avoid the conflicts.

Hii,

I have added the test case for this functionality. Let me know if any changes are needed.
Thanks!

Pipeline is passed now!

Assigning this to myself to resolve the phpch issues.

Hello,

Now all the phpstan issues are cleared. Thanks!

Hii,

I have created MR #36 for this feature request. Let me know if there are any changes needed.
Thanks!

Hii,

I have added the changes.
Thanks!

Hi @swirt,

There may be possibility that one content type can have multiple patterns also. In that case it will be good to show in table. Kindly share your view.
Thanks!

Hii,

I have created MR #4 to add this feature. Let me know if any changes are needed.

Thanks!

HI @lostcarpark,

Can I work on this?

Hi @swirt,

I have created MR #35 for this. Let me know if any changes are needed.
Thanks!

Hi @swirt,

It would be great if you can tell me the structure to display the data.

Thanks!

Hi @swirt,

I have created MR #34 to resolve this issue. Let me know if there are any changes needed.
Thanks!

Hi,

I have created MR #9 to resolve this issue. I have added the option in config form. Let me know if changes are needed.

nidhi27 → changed the visibility of the branch 3493506- to hidden.

Hi @swirt,

Got it your explanation. I observed that if menus are fieldable at that time its showing in the option.

For example, If you install menu_item_extras module at that time all the menu options are showing.

Kindly confirm is it expected behaviour or not.

Hi @swirt,

How it was working earlier?

Hii @swirt,

I have ceated MR for this. Let me know if any changes are need.

Thanks!

Hi @swirt,

Can you provide the more details on this ?

Hi @swirt,

I have implemented this section. I took the reference from core modules for this. Let me know if any changes are needed.
Thanks!

Hi @swirt,

Can you provide the step to reproduce this ?

Hi @swirt,

I have created MR !31 to resolve this issue. Let me know if any changes are needed.

Thanks!

Hi @pgrandeg,

Extremely sorry for the inconvenience. I missed to add the class in template. Now we can search by name or hexcode. I have committed the latest changes. Kindly look into it.

Thanks!

Hi @swirt,

I have addressed your comment. Thanks!

Hi @swirt,

I have created MR !30 for this issue. Let me know if any changes are needed.
Thanks!

Hi @swirt,

I have created MR !29 for this. I have implemented the 3rd approach for this. I am not sure whether it is correct method to implement this feature. It is not applicable for module and fields.
Let me know if any changes are needed.
Thanks!

Hi @swirt,

Can you please guide in which format data should be displayed?
Thanks!

Hi @swirt,

I have created MR !28 to add the reset button.

I have added the blue colour to filter button so that user can differentiate between two buttons. I have reloaded the form on clicking the reset button this will be helpful for https://www.drupal.org/project/content_model_documentation/issues/3386798 ✨ Make Field Search use query parameters as filter values Active .

Let me know if you have any other idea. Thanks!

Hi @swirt,

Do we need to add the reset button?

Hi @swirt,

I have created MR !26 for this feature request. Let me know if any changes are needed.
Thanks!

Hii,

I have created MR !12 to resolve this issue. Let me know if any other changes are needed.

Thanks!

Hii,

I have created MR !24 as per my understanding. Kindly review and let me know the changes.
Thanks!

Hii,

I have created MR !23 to resolve this issue.
Thanks!

nidhi27 → created an issue.

HI,

I have resolved the comments. Kindly review it.
Thanks!

HI,

I have resolved the comments and if paragraph module is not there, it is working s expected. Not throwing any error.

Thanks!

Hii,

I have created MR !22 for this issue. Let me know any changes are needed.
Thanks!

swirt → credited nidhi27 → .

Hii,

I have created MR #21 for this. Let me know if any changes are needed.
Thanks!

swirt → credited nidhi27 → .

HI @swirt,

I have created MR #25 with this request. Kindly review it and let me know.

Hi @swirt,

I have created MR #23 for this. Let me know if any changes are needed.
Thanks!

HI @swirt,

I have created MR #22 for media and block entity. Let me know if anything else is needed.
Thanks!

HI,

Is there any update on this feature request ?

Hi @anacolautti,

I have created MR #28 which replaces the jQuery code. It passed the eslint pipeline as well.
Thanks!

HI @opdavies,

Yes thats correct. It will be helpful if we add the test cases. But I dont have much idea about it so I wont be able to contribute here.
Thanks!

Hi @swirt,

I have created MR !20 for this feature request. Let me know if anything else is needed.
Thanks!

HI @swirt,

Let me know if I can work on this.
Thanks!

Hi @swirt,

I have crated MR #18 for this feature request. Let me know if any changes are needed. Thanks!

HI @pgrandeg,

I have checked the code sniffer changes and its working as expected. We can go ahead.

Hi,

Is there any update on this?

Hi @swirt,

Thanks for informing. I will look into it.

Hi @swentel,

I have added the update script to remove the permission and add the newer one. I have tested locally and its working as expected.
Thanks!

Hi @swentel,

Thats a good point.
I think we can run the update script where we can check if the permission is given on existing site, if yes than we will add that permission to all the comment type initially to run the site. Later on, it depends on the owner to change the permission.
Let me know your view on this. Thanks!