2 of your Email classes have a same email ID

  • Resolved lvtute

    (@lvtute)


    6 days, 16 hours ago

    Hi,

    I’m Shelby from YayMail dev team. We’re trying to integrate with WC Vendors and improve our addon.

    Till now we haven’t confronted any big problem with the integration since the code from your plugin is very well-written and clear (hats off to your Developers!).

    I only get a little issue that 2 of your email classes below is having the same email ID “admin_notify_application”:

    • Class WCVendors_Admin_Notify_Application from file wc-vendors/classes/admin/emails/class-wcv-admin-notify-application.php
    • Class WCVendors_Admin_Notify_Approved from file wc-vendors/classes/admin/emails/class-wcv-admin-notify-approved.php

    Of course we can do a workaround, but it’s a bit inconvenient. I wonder if it’s a mistake or you guys did it on purpose?

    I’m looking forward to hearing from you guys,

    Thank you.

    • This topic was modified 6 days, 16 hours ago by lvtute.
    • This topic was modified 6 days, 16 hours ago by lvtute.
Viewing 1 replies (of 1 total)
  • Plugin Support Fitri Sinta

    (@supportfitri)

    Hi @lvtute,

    Thank you for reaching out, and we really appreciate your kind words about our plugin! We’re glad to hear that the integration has been going smoothly for the most part.

    Regarding the issue you mentioned with the email classes, you’re absolutely right. We’ve identified this as a bug, and we’ll be including a fix in our next update to avoid any conflicts with the email IDs.

    We truly appreciate your patience and understanding, and we’ll keep you updated on the release. If you have any further questions or need assistance in the meantime, feel free to let us know!

    Best regards,

Viewing 1 replies (of 1 total)
  • You must be logged in to reply to this topic.