Kathmandu🇳🇵, Nepal
Account created on 25 July 2016, over 8 years ago
#

Recent comments

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi Sady, Thank you for reaching out and for your contributions to the module. I've been busy with other projects, so i couldn’t manage this one. I appreciate your work on the listed issues, and it’s great to hear the module is working well in production. I’m open to adding maintainers to the modules i’ve created. Thanks again for your efforts!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi Vladimir, Thank you for the update. Could you please try clearing the cache and check if the issue persists?

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The compatibility issue has been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The coding standard issue has been resolved in the latest version update.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The compatibility issue has been fixed in the latest version update.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal 11 compatibility fixes for leave_confirm have resolved the issues.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thanks Bram for the suggestion! I've removed the console.log as requested. Let me know if there's anything else.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi Peter, Apologies for the delay in getting back to you. The issue has been fixed. Please let me know if anything else comes up.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @coaston @sarwan_verma, Apologies for the late response! The Drupal 10 and 11 versions of the module are now available.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal 11 compatibility issues for have been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for the patch and guidelines. I have followed them and fixed the issues. If there are any remaining, please let me know.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for the patch and guidelines. I have followed them and fixed the issues. If there are any remaining, please let me know.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal 11 compatibility issues for have been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Apologies for the delayed response, and thank you for providing the patch. I’ve implemented it and addressed the issues reported by PHPCS.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thanks for the update, Erik! I’ve implemented the patch and it’s fixed. Let me know if anything else comes up.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi João, thanks for addressing the issue. I’ve reviewed it and removed the unnecessary JS file since it wasn’t needed. I appreciate the update, and let me know if anything else comes up.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

sujan shrestha made their first commit to this issue’s fork.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal 11 compatibility issue has been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The issues reported by PHPCS have been fixed. Thanks for the patch!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you! The issues reported by PHPCS have been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The compatibility issue has been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thanks for the patch! The issue has been resolved and everything is working perfectly now.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for the patch and your support. The issues reported by PHPCS have been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The compatibility issue has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi Karol, I apologize for the late response. I have merged the changes according to your suggestions. Thank you for the code and for bringing this issue to my attention. Please let me know if there's anything else that needs improvement.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

sujan shrestha made their first commit to this issue’s fork.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The compatibility issue has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for providing the patch and the guidelines to resolve the PHPCS issues. I have applied the fix, and I'm happy to confirm that the issue has been addressed in the latest version. Thanks again for your support!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi Pomelov, Apologies for the late response, and thank you for identifying the issue. The issue has been fixed in the latest version. Could you please update and check if the problem is resolved? Let me know if you need further help!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thanks David! Better Social Sharing Buttons and Better Social Share do seem to overlap in functionality, with both focusing on social sharing. However, Better Social Share offers more extensive customization, such as supporting over 100 social media platforms, including major networks like Facebook, Twitter, LinkedIn, WhatsApp, and Pinterest. It also allows for flexible integration into content entities, letting you configure sharing options from the manage display interface or embed sharing blocks on pages.

In terms of customization, Better Social Share gives you control over which icons are displayed, their order, and the ability to configure these settings with a "more" button that reveals additional social media icons, their placement, and other display settings. Plus, you can fine-tune the appearance of the icons with size and color adjustments, and use custom Twig for custom layouts, CSS, and JavaScript to enhance functionality.

These features might make Better Social Share more flexible compared to Better Social Sharing Buttons, depending on the level of customization needed. Additionally, I’d welcome any new feature suggestions or improvements that could enhance the product further!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal compatibility issue has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

I've moved this to 'Fixed' now that Patch #2 has been successfully implemented.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal compatibility issue has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal compatibility issue has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal compatibility issue has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi James, Thank you for the feedback, and I apologize for the delayed response. I’m planning to add the feature you mentioned, allowing the end user to specify the payment amount. I’ll let you know once it’s implemented.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for your feedback, and I apologize for the delayed response in addressing this issue. I’ve resolved the PHPCS-related issues. Thanks again!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for your feedback, and I apologize for the late response. I’ve updated the function names and also fixed the PHPCS-related issues.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi jcandan, Thank you for the feedback, and I apologize for the late response. I’ve updated the documentation as per your feedback regarding the composer require drupal/stripe_pay instructions. Thank you!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for the feedback, and I apologize for the delayed response. I’ve reviewed the feedback and the patch files, and I’ve fixed the issues reported by phpcs.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for the feedback, and I apologize for the delayed response. I’ve reviewed the feedback and the patch, and I’ve made the necessary changes to the documentation comments for the files, classes, functions, and methods. The issues have now been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @gbyte @aaron.ferris , Apologies for the late response, and thank you for your feedback! I’ve updated the stripe-payment.html.twig file so that the price is pulled from the field value instead of being hard-coded. I also made a change in the StripeController.php file, where I added $productPrice = $productPrice * 100; to convert the price into the smallest unit of the currency (like cents). Could you please review the changes and let me know if everything looks good?

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi tart0, Thanks for the feedback! Glad to hear the patch worked for the .yml file and you didn’t run into any problems. I agree, testing the more advanced features would be a good next step to confirm everything is working. Please test the module according to the "Getting Started" steps in the documentation from this link: https://www.drupal.org/project/stripe_pay . I’ll also be adding more details to the README file and updating the documentation. Let me know if you need any assistance!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @moolahtoshi, Thank you for reaching out regarding this issue. The error you're encountering has been addressed in the latest version. Please update to the latest version to resolve the problem. Additionally, I’m actively working on further improvements to enhance the overall experience. If you encounter any further issues or need assistance with the update, feel free to let me know.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The compatibility issue has been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @inst, Thank you for your kind words. I’ll keep releasing stable versions that follow Drupal’s security rules. Making sure the module stays secure and reliable is very important to me. Let me know if you have any other suggestions.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @w01f, Thank you so much for the kind words and feedback! Apologies for the delayed response. I really appreciate your suggestion about a horizontally-aligned version. I’ve actually been considering ways to implement this and am working on plans to make it feasible. I’ll keep this in mind as we continue to refine and improve the module.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The compatibility issue with Drupal 11 has been resolved in the latest version.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @bwoods, Thanks for the feedback! Glad you got it working with processed_text. Using that approach avoids the need for the raw filter, which is great for security. I’ve fixed the media issue for the text field with RendererInterface in the latest version. Please check it out and let me know if there’s anything else!

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thanks for the patch! I’ve implemented the changes accordingly.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @yhe @jaydev bhatt, Thanks for the update! I’ve moved the issue to fixed. If you come across any further issues or need additional guidance, please feel free to create a new issue.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @radheymkumar, Thanks for the patch! I’ve implemented the changes accordingly.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The compatibility and other issue has been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal 11 compatibility has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal 11 compatibility has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal 11 compatibility has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @w01f, Thanks for sharing your findings on the AddToAny module—it's great that you're looking into the third-party sharing and privacy implications. I’d be happy to weigh in or offer any perspective on this! From my experience, privacy concerns around social sharing modules are definitely something worth investigating closely. I’d be happy to help if you’d like, especially with other module options or privacy concerns. Let me know how I can contribute.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The problem with Drupal 11 compatibility has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @michelrrr, Thanks for letting me know about these issues.

The Pinterest button issue has been fixed in the dev version(1.1.x). Could you please test it on your end? If everything works fine, I'll go ahead and create a new version.

Regarding the problem with removing buttons in the configuration, could you please share a screenshot of the page? This will help me troubleshoot and fix the issue.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @w01f, thanks for sharing your thoughts! It's great to hear you're testing out the module on some new sites. I’d be really interested to hear about your experience and how it compares to AddtoAny once you've had a chance to dive in.

I also appreciate the mention of Better Social Sharing Buttons and the distinction between the two types of social sharing. It could definitely be helpful to clarify that for newcomers, so they better understand the difference between “Informative” sharing (linking to social media profiles) and “Sharing” (allowing users to share content directly).

The Better Social Share Drupal module enhances social sharing on the website without relying on third-party integrations like AddThis or AddToAny. It optimizes native sharing methods from popular social media platforms for a smoother, more efficient experience. With the supported social media platforms and extensive customization options, it helps expand reach and improve content sharing across online communities. Additionally, we can place the sharing blocks on pages or nodes to allow users to easily share the current page on social media.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for your patience and for pointing that out. After some thorough investigation and hard work, the fix has been implemented in version 2, and it’s now working correctly, including in the comments. Please feel free to test it on your end, and let me know.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

The Drupal 11 compatibility issue has been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Version 1.2.5 is compatible with Drupal 11.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for the automated Drupal 11 compatibility fixes. I’ve reviewed the changes and have made the necessary fixes for Drupal 11.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for the automated Drupal 11 compatibility fixes. I’ve reviewed the changes and have made the necessary fixes for Drupal 11.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for identifying the PHPCS issues. I’ve reviewed them, and the issues have been fixed.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for the merge request! I've successfully merged the changes.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

sujan shrestha made their first commit to this issue’s fork.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you @avpaderno for the information. I'll ensure the status is updated accordingly.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thanks for the feedback! I have added the missing README.md file.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thanks for the feedback! I've updated the README.md template accordingly.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Thank you for reaching out. Currently, we do not have plans to integrate with COOKiES Consent Management. However, we appreciate your interest and will keep your suggestion in mind for future updates.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Added the missing README.md file and resolved compatibility issues with Drupal 11.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @Joachim Namyslo Thank you for the bug report. I've adjusted the permissions for the route, and it appears to be working now. Could you please verify?

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @SirClickalot Thank you for reporting this issue! It has been fixed in version 1.1.4. Please update to that version and check if the problem persists.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @SirClickalot, Thank you for sharing your feedback and suggestions. I'll definitely take your feature requests into consideration and work on implementing them.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @wjackson @earthday47 Thank you for the patch! Your contribution is greatly appreciated. With this fix, we can now integrate the Feeds import with content that includes fields from the Simple Time Field module.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

sujan shrestha made their first commit to this issue’s fork.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @enchufe, thank you for the new feature request and for providing the patch for 'custom attributes to scripts' feature.

I have implemented the Xss::filter() function to properly sanitized before using raw to output the $script_attributes value. Here's how it's applied on the module file:

$script_attributes = Xss::filter($script_attributes);

Additionally, I've utilized the 'raw' filter like {{ script_attributes|raw }} on the twig file to display the HTML-encoded script attributes. For example, type="text/plain" data-category="analytics" was displayed as show on the attachment screenshot.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @ecvandenberg Thank you for reporting the issue. It seems that when unsubscribing a user by unchecking the checkbox for a specific list, the user isn't being unsubscribed from that list. I'll investigate further and provide updates accordingly.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @Promo-IL Thank you for letting me know. I'll review the code and ensure that everything aligns properly with the functionality. I appreciate your diligence in ensuring data integrity.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @FiNeX Thank you for sharing the patch! I appreciate your effort in addressing these issues. I'll make sure to review the patch thoroughly and get back to you soon.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

sujan shrestha made their first commit to this issue’s fork.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @delacosta456,

Thank you for your interest in the module. While my current module isn't part of the Social Initiative modules, I do have plans to develop a separate module that functions similarly to 'social_post_facebook'. This new module will automatically post content to Facebook pages or groups upon content creation or with a cron job, featuring options for random or queued content with specific time intervals.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

I've implemented the patch and the Drupal 10 compatibility issue has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @aman_lnwebworks @dineshkumarbollu , thank you for the feedback. I've implemented the patch, and the issue has been resolved.

🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

I have fixed the PHPCS issue on the 1.1.x branch and ran the command to verify that the issues have been resolved.

phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,info,txt,md,yml c
olor_pickr/
🇳🇵Nepal sujan shrestha Kathmandu🇳🇵, Nepal

Hi @ecvandenberg Thank you for your suggestion! I'll look into updating the email address in Moosend or, if that's not possible, unsubscribing the old email address. It's a valuable feature request, and I'm planning to address it.

Production build 0.71.5 2024