Removed emailErrors flag (#16900)

refs https://github.com/TryGhost/Team/issues/3337

Removed emailErrors flag as it is no longer needed

ghost/admin/app/services/feature.js

@@ -64,7 +64,6 @@ export default class FeatureService extends Service {
     @feature('lexicalMultiplayer') lexicalMultiplayer;
     @feature('audienceFeedback') audienceFeedback;
     @feature('webmentions') webmentions;
-    @feature('emailErrors') emailErrors;
     @feature('websockets') websockets;
     @feature('stripeAutomaticTax') stripeAutomaticTax;
     @feature('makingItRain') makingItRain;

ghost/admin/app/templates/settings/labs.hbs

@@ -228,19 +228,6 @@
                         </div>
                     </div>
                 </div>
-                <div class="gh-expandable-block">
-                    <div class="gh-expandable-header">
-                        <div>
-                            <h4 class="gh-expandable-title">Show email errors</h4>
-                            <p class="gh-expandable-description">
-                                This makes email errors visible in the UI.
-                            </p>
-                        </div>
-                        <div class="for-switch">
-                            <GhFeatureFlag @flag="emailErrors" />
-                        </div>
-                    </div>
-                </div>
                 <div class="gh-expandable-block">
                     <div class="gh-expandable-header">
                         <div>

ghost/core/core/server/api/endpoints/utils/serializers/output/mappers/emails.js

@@ -1,5 +1,3 @@
-const labs = require('../../../../../../../shared/labs');
-const config = require('../../../../../../../shared/config');
 const emailService = require('../../../../../../services/email-service');
 
 module.exports = (model, frame) => {
@@ -19,12 +17,6 @@ module.exports = (model, frame) => {
         }
     }
 
-    if (!labs.isSet('emailErrors') && !!(config.get('bulkEmail') && config.get('bulkEmail').mailgun)) {
-        if (jsonModel.status === 'failed') {
-            jsonModel.status = 'submitted';
-        }
-    }
-
     // Removed loaded post relation if set
     delete jsonModel.post;
 

ghost/core/core/shared/labs.js

@@ -17,7 +17,6 @@ const messages = {
 const GA_FEATURES = [
     'audienceFeedback',
     'themeErrorsNotification',
-    'emailErrors',
     'outboundLinkTagging',
     'announcementBar'
 ];

ghost/core/test/e2e-api/admin/__snapshots__/settings.test.js.snap

@@ -718,7 +718,7 @@ exports[`Settings API Edit Can edit a setting 2: [headers] 1`] = `
 Object {
   "access-control-allow-origin": "http://127.0.0.1:2369",
   "cache-control": "no-cache, private, no-store, must-revalidate, max-stale=0, post-check=0, pre-check=0",
-  "content-length": "4014",
+  "content-length": "3993",
   "content-type": "application/json; charset=utf-8",
   "content-version": StringMatching /v\\\\d\\+\\\\\\.\\\\d\\+/,
   "etag": StringMatching /\\(\\?:W\\\\/\\)\\?"\\(\\?:\\[ !#-\\\\x7E\\\\x80-\\\\xFF\\]\\*\\|\\\\r\\\\n\\[\\\\t \\]\\|\\\\\\\\\\.\\)\\*"/,