From 6bea6cba5d803f23c27c03f4b47310bd6ea744bb Mon Sep 17 00:00:00 2001 From: David Taylor Date: Fri, 13 May 2022 10:08:59 +0100 Subject: [PATCH] FIX: Add safari 12 to ember-cli build targets in production (#16745) cf273ec6 removed ie11 as a target. A side effect is that this also removed support for Safari 12, which we will be maintaining support for until January 2023 https://meta.discourse.org/t/224747 --- app/assets/javascripts/discourse/config/targets.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/app/assets/javascripts/discourse/config/targets.js b/app/assets/javascripts/discourse/config/targets.js index 6277c67845..13213df378 100644 --- a/app/assets/javascripts/discourse/config/targets.js +++ b/app/assets/javascripts/discourse/config/targets.js @@ -1,11 +1,19 @@ "use strict"; +const isCI = !!process.env.CI; +const isProduction = process.env.EMBER_ENV === "production"; + const browsers = [ "last 1 Chrome versions", "last 1 Firefox versions", "last 1 Safari versions", ]; +if (isCI || isProduction) { + // https://meta.discourse.org/t/224747 + browsers.push("Safari 12"); +} + module.exports = { browsers, };