From 8a52f9440fc42ea33685ff2e4c5bfe2602886523 Mon Sep 17 00:00:00 2001 From: Robin van der Linde Date: Wed, 27 Jul 2022 10:24:58 +0200 Subject: [PATCH] Dont actually mark task as fixed in dryrun --- UI/Popup/ImportButton.ts | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/UI/Popup/ImportButton.ts b/UI/Popup/ImportButton.ts index 83149e8a8..cdf299e7d 100644 --- a/UI/Popup/ImportButton.ts +++ b/UI/Popup/ImportButton.ts @@ -608,13 +608,16 @@ export class ImportPointButton extends AbstractImportButton { } let maproulette_id = originalFeatureTags.data[args.maproulette_id]; - console.log("Checking if we need to mark a maproulette challenge as fixed (" + maproulette_id + ")") + console.log("Checking if we need to mark a maproulette task as fixed (" + maproulette_id + ")") if (maproulette_id !== undefined) { - // Fetch MapRoulette API key, then use it to mark the challenge as fixed - console.log("Marking maproulette challenge as fixed") - state.maprouletteConnection.closeTask(Number(maproulette_id)); - originalFeatureTags.data["mr_taskStatus"] = "Fixed"; - originalFeatureTags.ping(); + if (state.featureSwitchIsTesting.data){ + console.log("Not marking maproulette task " + maproulette_id + " as fixed, because we are in testing mode") + } else { + console.log("Marking maproulette task as fixed") + state.maprouletteConnection.closeTask(Number(maproulette_id)); + originalFeatureTags.data["mr_taskStatus"] = "Fixed"; + originalFeatureTags.ping(); + } } }