Account created on 16 June 2023, over 1 year ago
#

Merge Requests

More

Recent comments

🇮🇳India roshni27

Hello,
I'm having trouble seeing this screenshot; could you provide additional details?

🇮🇳India roshni27

I apologize for the misunderstanding; please review this one.

🇮🇳India roshni27

Agreed, the deprecated method google.maps.event.addDomListener() has been successfully replaced, and the patch has been applied. Ready to move to RTBC.

🇮🇳India roshni27

Patch #70 has successfully passed the test in version 11.x, hence it should be moved to the "needs review"

🇮🇳India roshni27

Hey Webbeh,
the issue is active, and I'm on the same page as MR1. However, when I tested it in version 2.0.0, I noticed that in admin/config/media, the YouTube field setting label shows up. But when you navigate to the YouTube field setting tab, it's just the settings. So, I'd say MR1 is only halfway there. That's why I submitted a patch with screenshots. If you believe I don't deserve credit, feel free to remove it.
Thanks!

🇮🇳India roshni27

We can also proceed with this option.

🇮🇳India roshni27

Hi, I have created some logo designs. Please review them.

🇮🇳India roshni27

Hi, I have created some logo designs. Please review them.

🇮🇳India roshni27

Hello, I have created some logo designs. Please review them.

🇮🇳India roshni27

Hello,
I have created some logo designs. Please review them.

🇮🇳India roshni27

Hello,
I have created some logo designs. Please review them.

🇮🇳India roshni27

@ alex.druhet Thanks. The proposed logo updated according to #14.please review.

🇮🇳India roshni27

I have create some logo.Please review .

🇮🇳India roshni27

I've updated the code to enhance exception messages, so they now indicate whether the callback is null, empty, or missing, and also offer specific details about the type of callback failure. This improvement makes it simpler to identify and resolve issues with AJAX callbacks.if there is any mistake please guide me.

🇮🇳India roshni27

The exception message for an AJAX callback error has been modified in this patch. Please review.

🇮🇳India roshni27

Sorry for previous all. Resize image Please review.

🇮🇳India roshni27

Hello , I have refer #5 patch and create new patch please let me know any change , i will work on it.

🇮🇳India roshni27

The issue exists and has been resolved using the patch mentioned. Please refer to the screenshots for confirmation. +RTBC

🇮🇳India roshni27

Please review this description for pathauto:

Automatically generate aliases using criteria and tokens, with centralized settings for site administrators. Supports core entity types: content, taxonomy terms, users (including blogs and forum pages).Pathauto facilitates bulk alias deletion, benefits SEO, and enhances visitor usability.

🇮🇳India roshni27

I was able to reproduce the issue, and I attempted to solve it using group filters. However, I noticed that the values for 'enable' and 'disable' were set to 1 and 2, which is incorrect. We actually need the values to be 0 for 'disable' and 1 for 'enable.' I have attached a screenshot for reference. If I have misunderstood, please correct me.

🇮🇳India roshni27

Thanks @bwaindwain
The issue existed and was resolved after applying the patch. Please refer to the screenshot for details.
+1 RTBC .

🇮🇳India roshni27

"I cannot reproduce it. Please refer to the screenshot."

🇮🇳India roshni27

I agree with you, Sakthi. I have provide a patch and a screenshot for reference. Kindly review.

🇮🇳India roshni27

@Berdir Please suggest me what logic we should implement here when $entity_id is null?

🇮🇳India roshni27

This code should handle various cases, including when entity_id is null or not a valid string.

🇮🇳India roshni27

The issue has been addressed and resolved by the MR above.So move to RTBC.

🇮🇳India roshni27

This issue exists and has been resolved by the MR above; please refer to the screenshot (SS) for more details.

🇮🇳India roshni27

Thank you for your feedback! I appreciate your guidance on the issue categorization. I'll make sure to focus on non-novice issues going forward.

🇮🇳India roshni27

Thank you for your feedback! I appreciate your guidance on the issue categorization. I'll make sure to focus on non-novice issues going forward.

🇮🇳India roshni27

I have successfully installed Easy Install in Drupal 10 using MR 5, and it is currently working as expected for me. Please refer to the screenshot below for confirmation

🇮🇳India roshni27

Based on the comments and suggestions #32, here's the updated code with more explicit checks for empty strings.

🇮🇳India roshni27

I have applied the patch mentioned above, but it seems that it's not compatible with Drupal 10, as indicated in the screenshot. I am working on resolving them and will upload a new patch once done.

🇮🇳India roshni27

Call to deprecated function drupal_uninstall_schema()
. Deprecated in drupal:9.2.0 and is removed from drupal:10.0.0. No direct replacement is provided.
So I have remove it and provide patch please review.

🇮🇳India roshni27

I have applied the above patch. It have found 2 warnings indicated in the screenshot. I am working on resolving them and will upload a new patch once done.

🇮🇳India roshni27

@joachim yes you are right. I have update it .please review.

Production build 0.71.5 2024