Please review this.
I am working on this.
roshni27 → made their first commit to this issue’s fork.
Hello,
I'm having trouble seeing this screenshot; could you provide additional details?
I apologize for the misunderstanding; please review this one.
roshni27 → created an issue.
Agreed, the deprecated method google.maps.event.addDomListener() has been successfully replaced, and the patch has been applied. Ready to move to RTBC.
please review paatch
I have implemented the modifications as per issue #6. please review the patch.
Patch #70 has successfully passed the test in version 11.x, hence it should be moved to the "needs review"
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!
Please review MR.
roshni27 → created an issue.
please review.
Refer SS for labels inconsistent.
@bserem please review.
@BramDriesen Thanks. please review.
please review
please review
please review
We can also proceed with this option.
Hi, I have created some logo designs. Please review them.
Hi, I have created some logo designs. Please review them.
please review.
Hello, I have created some logo designs. Please review them.
Hello,
I have created some logo designs. Please review them.
Hello,
I have created some logo designs. Please review them.
@ alex.druhet Thanks. The proposed logo updated according to #14.please review.
please review.
please review.
please review.
please review
Ok. I will update
Please review.
roshni27 → made their first commit to this issue’s fork.
I have create some logo.Please review .
Please review.
please review.
please review.
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.
please review.
The exception message for an AJAX callback error has been modified in this patch. Please review.
Please review.
please review.
Sorry for previous all. Resize image Please review.
Please review .
Please review the logo.
Please review the logo.
Please review the patch.
I am working on it.
Hello , I have refer #5 patch and create new patch please let me know any change , i will work on it.
The issue exists and has been resolved using the patch mentioned. Please refer to the screenshots for confirmation. +RTBC
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.
Please review the MR.
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.
Thanks @bwaindwain
The issue existed and was resolved after applying the patch. Please refer to the screenshot for details.
+1 RTBC .
Please review .
"I cannot reproduce it. Please refer to the screenshot."
I agree with you, Sakthi. I have provide a patch and a screenshot for reference. Kindly review.
@Berdir Please suggest me what logic we should implement here when $entity_id is null?
This code should handle various cases, including when entity_id is null or not a valid string.
Updated patch please review.
Please review the patch.
roshni27 → created an issue.
Please review the patch.
Please review MR.
roshni27 → created an issue.
The issue has been addressed and resolved by the MR above.So move to RTBC.
I will review it.
Please review the patch.
This issue exists and has been resolved by the MR above; please refer to the screenshot (SS) for more details.
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.
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.
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
same patch against 11.x
I have made changes according to #15 and #16.
Based on the comments and suggestions #32, here's the updated code with more explicit checks for empty strings.
Please review it.
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.
Please review the patch.
I working on it.
roshnichordiya → created an issue.
roshnichordiya → created an issue.
Please review
roshnichordiya → created an issue. See original summary → .
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.
Please review.
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.
@joachim yes you are right. I have update it .please review.