From 6741f139d51bf6765451c70d82574a5332c2505f Mon Sep 17 00:00:00 2001 From: Hannah Wolfe Date: Fri, 9 Sep 2022 15:49:29 +0100 Subject: [PATCH] Updated redirects todo to be a deprecation notice - When we have todos related to deprecations, we should use @deprecated instead - @deprecated notices should say when a feature was deprecated, not when it was removed --- ghost/core/core/server/api/endpoints/redirects.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ghost/core/core/server/api/endpoints/redirects.js b/ghost/core/core/server/api/endpoints/redirects.js index 8393d9b4d2..09295969b0 100644 --- a/ghost/core/core/server/api/endpoints/redirects.js +++ b/ghost/core/core/server/api/endpoints/redirects.js @@ -12,9 +12,7 @@ module.exports = { value() { return redirects.api.getRedirectsFilePath() .then((filePath) => { - // TODO: Default file type is .json for backward compatibility. - // When .yaml becomes default or .json is removed at v4, - // This part should be changed. + // @deprecated: .json was deprecated in v4.0 but is still the default for backwards compat return filePath === null || path.extname(filePath) === '.json' ? 'redirects.json' : 'redirects.yaml';