From 4e43e9b2fdaa3fd61ba835046145274c476b513c Mon Sep 17 00:00:00 2001 From: Oleg Isonen Date: Fri, 5 Jan 2024 17:41:31 +0100 Subject: [PATCH] fix: blocking alerts accidentally got enabled for everyone (#2731) ## Description In the last PR I forgot to uncomment the check ## Steps for reproduction 1. click button 2. expect xyz ## Code Review - [ ] hi @kof, I need you to do - conceptual review (architecture, feature-correctness) - detailed review (read every line) - test it on preview ## Before requesting a review - [ ] made a self-review - [ ] added inline comments where things may be not obvious (the "why", not "what") ## Before merging - [ ] tested locally and on preview environment (preview dev login: 5de6) - [ ] updated [test cases](https://github.com/webstudio-is/webstudio/blob/main/apps/builder/docs/test-cases.md) document - [ ] added tests - [ ] if any new env variables are added, added them to `.env.example` and the `builder/env-check.js` if mandatory --- .../app/builder/features/blocking-alerts/blocking-alerts.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/builder/app/builder/features/blocking-alerts/blocking-alerts.tsx b/apps/builder/app/builder/features/blocking-alerts/blocking-alerts.tsx index a12ebf9341..02535ada2b 100644 --- a/apps/builder/app/builder/features/blocking-alerts/blocking-alerts.tsx +++ b/apps/builder/app/builder/features/blocking-alerts/blocking-alerts.tsx @@ -25,7 +25,7 @@ const useUnsupportedBrowser = () => { const [message, setMessage] = useState(); useEffect(() => { if ("chrome" in window || isFeatureEnabled("unsupportedBrowsers")) { - //return; + return; } setMessage(