Merge pull request #5611 from overleaf/revert-4746-ae-project-history

Revert "Add endpoint for migrating a project from track-changes to full project history"

GitOrigin-RevId: 4a37d31daa2855622bfd3e7d96eeabe813b5d8bb

services/web/.gitignore

@@ -80,6 +80,3 @@ modules/**/Makefile
 # Intellij
 .idea
 .run
-
-# Test fixture zip
-!modules/admin-panel/test/unit/src/data/track-changes-project.zip

services/web/Dockerfile

@@ -30,7 +30,6 @@ RUN mkdir -p /app/data/dumpFolder && \
   mkdir -p /app/data/pdf && \
   mkdir -p /app/data/uploads && \
   mkdir -p /app/data/zippedProjects && \
-  mkdir -p /app/data/projectHistories && \
   chmod -R 0755 /app/data/ && \
   chown -R node:node /app/data/
 

services/web/app/src/Features/History/HistoryManager.js

@@ -9,7 +9,6 @@ module.exports = {
   flushProject: callbackify(flushProject),
   resyncProject: callbackify(resyncProject),
   deleteProject: callbackify(deleteProject),
-  deleteProjectHistory: callbackify(deleteProjectHistory),
   injectUserDetails: callbackify(injectUserDetails),
   promises: {
     initializeProject,
@@ -17,7 +16,6 @@ module.exports = {
     resyncProject,
     deleteProject,
     injectUserDetails,
-    deleteProjectHistory,
   },
 }
 
@@ -57,23 +55,10 @@ async function flushProject(projectId) {
   }
 }
 
-async function deleteProjectHistory(projectId) {
-  try {
-    await request.delete({
-      url: `${settings.apis.project_history.url}/project/${projectId}`,
-    })
-  } catch (err) {
-    throw OError.tag(err, 'failed to delete project history', {
-      projectId,
-    })
-  }
-}
-
-async function resyncProject(projectId, force = false) {
+async function resyncProject(projectId) {
   try {
     await request.post({
       url: `${settings.apis.project_history.url}/project/${projectId}/resync`,
-      qs: { force }, // TODO: only send if true?
     })
   } catch (err) {
     throw OError.tag(err, 'failed to resync project history', { projectId })

services/web/app/src/Features/Project/ProjectHistoryHandler.js

@@ -70,15 +70,7 @@ const ProjectHistoryHandler = {
     )
   },
 
-  unsetHistory(project_id, callback) {
-    return Project.updateOne(
-      { _id: project_id },
-      { $unset: { 'overleaf.history': true } },
-      callback
-    )
-  },
-
-  upgradeHistory(project_id, allowDowngrade, callback) {
+  upgradeHistory(project_id, callback) {
     // project must have an overleaf.history.id before allowing display of new history
     if (callback == null) {
       callback = function () {}
@@ -88,7 +80,6 @@ const ProjectHistoryHandler = {
       {
         'overleaf.history.display': true,
         'overleaf.history.upgradedAt': new Date(),
-        'overleaf.history.allowDowngrade': allowDowngrade,
       },
       function (err, result) {
         if (err != null) {

services/web/app/src/models/Project.js

@@ -99,7 +99,6 @@ const ProjectSchema = new Schema({
       id: { type: Number },
       display: { type: Boolean },
       upgradedAt: { type: Date },
-      allowDowngrade: { type: Boolean },
     },
   },
   collabratecUsers: [

services/web/package.json

@@ -169,7 +169,6 @@
     "sanitize-html": "^1.27.1",
     "scroll-into-view-if-needed": "^2.2.25",
     "underscore": "^1.13.1",
-    "unzipper": "^0.10.11",
     "url-parse": "^1.4.7",
     "utf-8-validate": "^5.0.2",
     "uuid": "^3.0.1",
@@ -242,7 +241,6 @@
     "mkdirp": "0.5.1",
     "mocha": "^8.3.2",
     "mock-fs": "^4.11.0",
-    "nock": "^13.1.1",
     "node-fetch": "^2.6.1",
     "nodemon": "^2.0.6",
     "optimize-css-assets-webpack-plugin": "^5.0.3",