feat: add an admin notice when ses email is disabled because of the vanity domain

carlalexander · carlalexander · commit 55d7703ef054 · 2024-09-05T17:17:14.000-04:00
diff --git a/pluggable.php b/pluggable.php
@@ -24,7 +24,11 @@
     add_action('admin_notices', function () {
         echo '<div class="notice notice-warning"><p><strong>Ymir:</strong> Sending emails using SES is disabled because the "wp_mail" function was already overridden by another plugin.</p></div>';
     });
-} elseif ($ymir->isSesEnabled() && !function_exists('wp_mail')) {
+} elseif ($ymir->isSesEnabled() && $ymir->isUsingVanityDomain()) {
+    add_action('admin_notices', function () {
+        echo '<div class="notice notice-warning"><p><strong>Ymir:</strong> Sending emails using SES is disabled because the site is using a vanity domain. To learn how to map a domain to your environment, check out <a href="https://docs.ymirapp.com/guides/domain-mapping.html">this guide</a>.</p></div>';
+    });
+} elseif ($ymir->isSesEnabled() && !$ymir->isUsingVanityDomain() && !function_exists('wp_mail')) {
     /**
      * Send email using the cloud provider email client.
      */
diff --git a/src/Plugin.php b/src/Plugin.php
@@ -112,11 +112,18 @@ public function isLoaded(): bool
      */
     public function isSesEnabled(): bool
     {
-        return !$this->container['ymir_using_vanity_domain']
-            && false === getenv('YMIR_DISABLE_SES')
+        return false === getenv('YMIR_DISABLE_SES')
             && (!defined('YMIR_DISABLE_SES') || !YMIR_DISABLE_SES);
     }
 
+    /**
+     * Checks if the site is using a vanity domain.
+     */
+    public function isUsingVanityDomain(): bool
+    {
+        return $this->container['ymir_using_vanity_domain'];
+    }
+
     /**
      * Loads the plugin into WordPress.
      */
