yank outdated redirect code

Author: sarahs

lib/redirects/precompile.js

@@ -50,22 +50,6 @@ module.exports = async function precompileRedirects (pageList, pageMap) {
     const developerRouteWithLanguage = `/en${developerRoute}`
     allRedirects[developerRouteWithLanguage] = newPath
 
-    // TODO until we update all the old /v3 and /v4 links, we need to support redirects
-    // from the old /enterprise/<number>/v3 format to the new /enterprise-server@<number/rest format
-    // AS WELL AS /enterprise-server@<number/v3 to /enterprise-server@<number/rest.
-    // This is because the new format gets created dynamically even when the links point to /v3 or /v4.
-    // EXAMPLES:
-    // /en/enterprise/2.20/v3/pulls/comments -> /en/enterprise-server@2.20/rest/reference/pulls#comments
-    // /en/enterprise-server@2.20/v3/pulls/comments -> /en/enterprise-server@2.20/rest/reference/pulls#comments
-    // NOTE: after we update all the /v3 and /v4 links, we can yank the following block
-    if (developerRoute.includes('/enterprise/')) {
-      const developerRouteWithNewFormat = developerRoute.replace(/\/enterprise\/(\d.\d\d)\//, '/enterprise-server@$1/')
-      const developerRouteWithNewFormatWithLanguage = `/en${developerRouteWithNewFormat}`
-      allRedirects[developerRouteWithNewFormat] = newPath
-      allRedirects[developerRouteWithNewFormatWithLanguage] = newPath
-    }
-    // TODO ENDYANK
-
     // although we only support developer Enterprise paths up to 2.21, we make
     // an exception to always redirect versionless paths to the latest version
     if (developerRoute.includes('/2.21/')) {
@@ -74,32 +58,7 @@ module.exports = async function precompileRedirects (pageList, pageMap) {
       const developerRouteWithLanguageWithoutVersion = `/en${developerRouteWithoutVersion}`
       allRedirects[developerRouteWithoutVersion] = newPathOnLatestVersion
       allRedirects[developerRouteWithLanguageWithoutVersion] = newPathOnLatestVersion
-      // TODO after we update all the /v3 and /v4 links, we can yank the following
-      const developerRouteWithoutVersionWithNewFormat = developerRouteWithoutVersion
-        .replace('/enterprise/', 'enterprise-server')
-      const developerRouteWithoutVersionWithNewFormatWithLanguage = `/en${developerRouteWithoutVersionWithNewFormat}`
-      allRedirects[developerRouteWithoutVersionWithNewFormat] = newPathOnLatestVersion
-      allRedirects[developerRouteWithoutVersionWithNewFormatWithLanguage] = newPathOnLatestVersion
-      // TODO ENDYANK
-    }
-
-    // TODO: TEMPORARILY support explicit 2.22 redirects (created on page render by lib/rewrite-local-links)
-    // after we update `/v3` and `/v4` links everywhere to `/rest` and `/graphql`, we can
-    // yank this entire block because 2.22 never existed on developer site
-    if (developerRoute.includes('/2.21/')) {
-      const newPath222 = newPath.replace('@2.21/', '@2.22/')
-      const developerRoute222 = developerRoute.replace('/2.21/', '/2.22/')
-      const developerRouteWithLanguage222 = `/en${developerRoute222}`
-      allRedirects[developerRoute222] = newPath222
-      allRedirects[developerRouteWithLanguage222] = newPath222
-
-      const developerRouteWithNewFormat222 = developerRoute222
-        .replace('/enterprise/2.22/', '/enterprise-server@2.22/')
-      const developerRouteWithNewFormatWithLanguage222 = `/en${developerRouteWithNewFormat222}`
-      allRedirects[developerRouteWithNewFormat222] = newPath222
-      allRedirects[developerRouteWithNewFormatWithLanguage222] = newPath222
     }
-    // TODO ENDYANK
 
     // given a developer route like `/enterprise/2.19/v3/activity`,
     // add a veriation like `/enterprise/2.19/user/v3/activity`;