From 10a2e21636e93d2cf2416ee1abe9cfc3b5e2dc60 Mon Sep 17 00:00:00 2001
From: Elian Doran <contact@eliandoran.me>
Date: Sat, 18 Apr 2026 19:29:54 +0300
Subject: [PATCH] chore(etapi): revert change to revision API

---
 apps/server/spec/etapi/post-revision.spec.ts | 9 +++------
 apps/server/src/etapi/notes.ts               | 4 ++--
 2 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/apps/server/spec/etapi/post-revision.spec.ts b/apps/server/spec/etapi/post-revision.spec.ts
index 3714132fa2..6c17c88384 100644
--- a/apps/server/spec/etapi/post-revision.spec.ts
+++ b/apps/server/spec/etapi/post-revision.spec.ts
@@ -1,5 +1,5 @@
 import { Application } from "express";
-import { beforeAll, describe, expect, it } from "vitest";
+import { beforeAll, describe, it } from "vitest";
 import supertest from "supertest";
 import { createNote, login } from "./utils.js";
 import config from "../../src/services/config.js";
@@ -20,13 +20,10 @@ describe("etapi/post-revision", () => {
     });
 
     it("posts note revision", async () => {
-        const response = await supertest(app)
+        await supertest(app)
             .post(`/etapi/notes/${createdNoteId}/revision`)
             .auth(USER, token, { "type": "basic"})
             .send("Changed content")
-            .expect(201);
-
-        expect(response.body.revisionId).toBeTruthy();
-        expect(response.body.noteId).toBe(createdNoteId);
+            .expect(204);
     });
 });
diff --git a/apps/server/src/etapi/notes.ts b/apps/server/src/etapi/notes.ts
index 8edc583435..9949b7abd2 100644
--- a/apps/server/src/etapi/notes.ts
+++ b/apps/server/src/etapi/notes.ts
@@ -193,9 +193,9 @@ function register(router: Router) {
         const note = eu.getAndCheckNote(req.params.noteId);
 
         const description = req.body?.description || "";
-        const revision = note.saveRevision({ description, source: "etapi" });
+        note.saveRevision({ description, source: "etapi" });
 
-        res.status(201).json(mappers.mapRevisionToPojo(revision));
+        return res.sendStatus(204);
     });
 
     eu.route<{ noteId: string }>(router, "get", "/etapi/notes/:noteId/attachments", (req, res, next) => {