From aa5fc97269790f74bc41d1b63bc2b283459bf211 Mon Sep 17 00:00:00 2001
From: pietervdvn <pietervdvn@posteo.net>
Date: Wed, 24 Aug 2022 01:29:11 +0200
Subject: [PATCH] Add warning for big minzoom

---
 Models/ThemeConfig/Conversion/Validation.ts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Models/ThemeConfig/Conversion/Validation.ts b/Models/ThemeConfig/Conversion/Validation.ts
index f6a877df8a..44b1763f67 100644
--- a/Models/ThemeConfig/Conversion/Validation.ts
+++ b/Models/ThemeConfig/Conversion/Validation.ts
@@ -537,6 +537,10 @@ export class ValidateLayer extends DesugaringStep<LayerConfigJson> {
                 errors
             }
         }
+        
+        if(json.minzoom > Constants.userJourney.minZoomLevelToAddNewPoints ){
+            (json.presets?.length > 0 ? errors : warnings).push(`At ${context}: minzoom is ${json.minzoom}, this should be at most ${Constants.userJourney.minZoomLevelToAddNewPoints} as a preset is set. Why? Selecting the pin for a new item will zoom in to level before adding the point. Having a greater minzoom will hide the points, resulting in possible duplicates`)
+        }
 
         {
             // duplicate ids in tagrenderings check