@raketa-cloud/admin 1.2.0 → 1.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raketa-cloud/admin",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "main": "src/index.js",
   "files": [
     "src/",

package/src/HonoAdapter.js

@@ -36,10 +36,7 @@ export default class HonoAdapter extends Adapter {
     this.globalConfig = config;
 
     // TODO: Pass in a path to redirect to after login?
-    setupAuthentication(
-      router,
-      config
-    );
+    setupAuthentication(router, config);
 
     router.get("/", (c) => {
       return c.redirect("/en");
@@ -101,6 +98,7 @@ export default class HonoAdapter extends Adapter {
         ctx.resourceConfig = resourceConfig;
         ctx.locales = this.globalConfig.i18n.locales;
         ctx.defaultLocale = this.globalConfig.i18n.defaultLocale;
+        ctx.maxOpenRevisions = this.globalConfig?.maxOpenRevisions || 3;
         ctx.locale = currentLocale;
         ctx.remoteConfig = { ...this.globalConfig.remoteConfig };
         ctx.logger = Logger;
@@ -110,6 +108,7 @@ export default class HonoAdapter extends Adapter {
           locales: ctx.locales,
           locale: ctx.locale,
           defaultLocale: ctx.defaultLocale,
+          maxOpenRevisions: ctx.maxOpenRevisions,
           path: ctx.req.path,
           query: ctx.query || {},
           layout: {

package/src/handlers/article_revisions/ArticleRevisionsHandler.js

@@ -1,6 +1,14 @@
 const create = async (ctx) => {
   const { parent_id } = ctx.params;
 
+  const article = await ctx.client.articles.find(parent_id, ctx.locale);
+
+  if (article.revisionsCount >= ctx.maxOpenRevisions) {
+    return {
+      redirect: `/${ctx.locale}/${ctx.resourceConfig.articleTypePlural}/${parent_id}?revisions_count_error=1`,
+    };
+  }
+
   const { errors, articleErrors } = await ctx.client.articleRevisions.create(
     parent_id,
     ctx.locale

package/src/handlers/articles/ArticlesHandler.js

@@ -26,7 +26,18 @@ const show = async (ctx) => {
   }
 
   const revisions = await ctx.client.articleRevisions.all(id, ctx.locale);
-  const changes = await ctx.client.articleChanges.list({}, id, ctx.locale);
+  const changes = await ctx.client.articleChanges.list(
+    {
+      "f:change_type": [
+        "revision_publish",
+        "revision_lock",
+        "revision_unlock",
+        "update",
+      ],
+    },
+    id,
+    ctx.locale
+  );
 
   return {
     assigns: {

package/src/handlers/event_revisions/EventRevisionsHandler.js

@@ -1,6 +1,14 @@
 const create = async (ctx) => {
   const { parent_id } = ctx.params;
 
+  const event = await ctx.client.events.find(parent_id, ctx.locale);
+
+  if (event.revisionsCount >= ctx.maxOpenRevisions) {
+    return {
+      redirect: `/${ctx.locale}/${ctx.resourceConfig.eventTypePlural}/${parent_id}?revisions_count_error=1`,
+    };
+  }
+
   const { errors, eventErrors } = await ctx.client.eventRevisions.create(
     parent_id,
     ctx.locale

package/src/handlers/events/EventsHandler.js

@@ -1,4 +1,4 @@
-import { toDateTimeInput, toDateInput } from "../../utils";
+import { toDateInput, toDateTimeInput } from "../../utils";
 
 const index = async (ctx) => {
   const items = await ctx.client.events.list(
@@ -26,7 +26,18 @@ const show = async (ctx) => {
   }
 
   const revisions = await ctx.client.eventRevisions.all(id, ctx.locale);
-  const changes = await ctx.client.eventChanges.list({}, id, ctx.locale);
+  const changes = await ctx.client.eventChanges.list(
+    {
+      "f:change_type": [
+        "revision_publish",
+        "revision_lock",
+        "revision_unlock",
+        "update",
+      ],
+    },
+    id,
+    ctx.locale
+  );
 
   return {
     assigns: {

package/src/handlers/page_revisions/PageRevisionsHandler.js

@@ -1,6 +1,14 @@
 const create = async (ctx) => {
   const { parent_id } = ctx.params;
 
+  const page = await ctx.client.pages.find(parent_id, ctx.locale);
+
+  if (page.revisionsCount >= ctx.maxOpenRevisions) {
+    return {
+      redirect: `/${ctx.locale}/${ctx.resourceConfig.pageTypePlural}/${parent_id}?revisions_count_error=1`,
+    };
+  }
+
   const { errors, pageErrors } = await ctx.client.pageRevisions.create(
     parent_id,
     ctx.locale

package/src/handlers/pages/PagesHandler.js

@@ -25,7 +25,18 @@ const show = async (ctx) => {
   }
 
   const revisions = await ctx.client.pageRevisions.all(id, ctx.locale);
-  const changes = await ctx.client.pageChanges.list({}, id, ctx.locale);
+  const changes = await ctx.client.pageChanges.list(
+    {
+      "f:change_type": [
+        "revision_publish",
+        "revision_lock",
+        "revision_unlock",
+        "update",
+      ],
+    },
+    id,
+    ctx.locale
+  );
 
   return {
     assigns: {

package/src/tests/handlers/article_revisions/ArticleRevisionsHandler.test/create.test.js

@@ -1,12 +1,13 @@
-import ArticleRevisionsHandler from "../../../../handlers/article_revisions/ArticleRevisionsHandler";
 import ArticleRevisions from "../../../../cloud/ArticleRevisions";
-
+import Articles from "../../../../cloud/Articles";
+import ArticleRevisionsHandler from "../../../../handlers/article_revisions/ArticleRevisionsHandler";
 import Factory from "../../../support/factory";
 
 function buildCtx(articleID) {
   return {
     client: {
       articleRevisions: new ArticleRevisions(() => ({}), "token"),
+      articles: new Articles(() => ({}), "token"),
     },
     params: { parent_id: articleID },
     locale: "en",
@@ -14,66 +15,73 @@ function buildCtx(articleID) {
       articleType: "article",
       articleTypePlural: "articles",
     },
+    maxOpenRevisions: 1,
   };
 }
 
 describe("Article revisions", () => {
   test("create successful", async () => {
+    const article = Factory.build("article", { id: 1, revisionsCount: 0 });
     const revision = Factory.build("article_revision", { originalId: 1 });
-    const ctx = buildCtx(revision.originalId);
+    const ctx = buildCtx(article.id);
+
+    jest.spyOn(ctx.client.articles, "find").mockResolvedValue(article);
 
-    const mockCreate = jest
+    jest
       .spyOn(ctx.client.articleRevisions, "create")
-      .mockImplementation(() => ({
-        revision,
-        errors: null,
-        articleErrors: null,
-      }));
+      .mockResolvedValue({ revision, errors: null, articleErrors: null });
 
     const result = await ArticleRevisionsHandler.create(ctx);
 
-    expect(mockCreate).toHaveBeenCalledWith(revision.originalId, "en");
+    expect(result).toEqual({
+      redirect: `/en/articles/${article.id}`,
+    });
+  });
+
+  test("create failed when there is an opened revision for the article", async () => {
+    const article = Factory.build("article", { id: 1, revisionsCount: 2 });
+    const ctx = buildCtx(article.id);
+
+    jest.spyOn(ctx.client.articles, "find").mockResolvedValue(article);
+
+    const result = await ArticleRevisionsHandler.create(ctx);
 
     expect(result).toEqual({
-      redirect: `/en/articles/${revision.originalId}`,
+      redirect: `/en/articles/${article.id}?revisions_count_error=1`,
     });
   });
 
   test("create failed with errors", async () => {
-    const ctx = buildCtx(1);
+    const article = Factory.build("article", { id: 1, revisionsCount: 0 });
     const errors = [{ key: "generalError", message: "something went wrong" }];
+    const ctx = buildCtx(article.id);
 
-    const mockCreate = jest
-      .spyOn(ctx.client.articleRevisions, "create")
-      .mockImplementation(() => ({
-        revision: null,
-        errors,
-        articleErrors: null,
-      }));
+    jest.spyOn(ctx.client.articles, "find").mockResolvedValue(article);
 
-    expect(async () => {
-      await ArticleRevisionsHandler.create(ctx);
-    }).rejects.toThrow(`Failed to start revision: ${JSON.stringify(errors)}`);
+    jest
+      .spyOn(ctx.client.articleRevisions, "create")
+      .mockResolvedValue({ revision: null, errors, articleErrors: null });
 
-    expect(mockCreate).toHaveBeenCalledWith(1, "en");
+    await expect(ArticleRevisionsHandler.create(ctx)).rejects.toThrow(
+      `Failed to start revision: ${JSON.stringify(errors)}`
+    );
   });
 
   test("create failed with article errors", async () => {
-    const ctx = buildCtx(1);
+    const article = Factory.build("article", { id: 1, revisionsCount: 0 });
     const errors = [{ key: "generalError", message: "something went wrong" }];
+    const ctx = buildCtx(article.id);
 
-    const mockCreate = jest
-      .spyOn(ctx.client.articleRevisions, "create")
-      .mockImplementation(() => ({
-        revision: null,
-        errors: null,
-        articleErrors: errors,
-      }));
+    jest.spyOn(ctx.client.articles, "find").mockResolvedValue(article);
 
-    expect(async () => {
-      await ArticleRevisionsHandler.create(ctx);
-    }).rejects.toThrow(`Failed to start revision: ${JSON.stringify(errors)}`);
+    jest.spyOn(ctx.client.articleRevisions, "create").mockResolvedValue({
+      revision: null,
+      errors: null,
+      articleErrors: errors,
+    });
 
-    expect(mockCreate).toHaveBeenCalledWith(1, "en");
+    await expect(ArticleRevisionsHandler.create(ctx)).rejects.toThrow(
+      `Failed to start revision: ${JSON.stringify(errors)}`
+    );
   });
 });

package/src/tests/handlers/articles/ArticlesHandler.test/show.test.js

@@ -72,7 +72,18 @@ describe("Articles Handler", () => {
 
     expect(mockedArticleFind).toHaveBeenCalledWith(article.id, "en");
     expect(mockedArticleRevisions).toHaveBeenCalledWith(article.id, "en");
-    expect(mockedArticleChanges).toHaveBeenCalledWith({}, article.id, "en");
+    expect(mockedArticleChanges).toHaveBeenCalledWith(
+      {
+        "f:change_type": [
+          "revision_publish",
+          "revision_lock",
+          "revision_unlock",
+          "update",
+        ],
+      },
+      article.id,
+      "en"
+    );
 
     expect(result).toEqual({
       assigns: {

package/src/tests/handlers/event_revisions/EventRevisionsHandler.test/create.test.js

@@ -1,4 +1,6 @@
+import Events from "../../../../cloud/Events";
 import EventRevisionsHandler from "../../../../handlers/event_revisions/EventRevisionsHandler";
+
 import EventRevisions from "../../../../cloud/EventRevisions";
 
 import Factory from "../../../support/factory";
@@ -7,6 +9,7 @@ function buildCtx(eventID) {
   return {
     client: {
       eventRevisions: new EventRevisions(() => ({}), "token"),
+      events: new Events(() => ({}), "token"),
     },
     params: { parent_id: eventID },
     locale: "en",
@@ -14,66 +17,71 @@ function buildCtx(eventID) {
       eventType: "event",
       eventTypePlural: "events",
     },
+    maxOpenRevisions: 1,
   };
 }
 
 describe("Event revisions", () => {
   test("create successful", async () => {
+    const event = Factory.build("event", { id: 1, revisionsCount: 0 });
     const revision = Factory.build("event_revision", { originalId: 1 });
-    const ctx = buildCtx(revision.originalId);
+    const ctx = buildCtx(event.id);
+
+    jest.spyOn(ctx.client.events, "find").mockResolvedValue(event);
 
-    const mockCreate = jest
+    jest
       .spyOn(ctx.client.eventRevisions, "create")
-      .mockImplementation(() => ({
-        revision,
-        errors: null,
-        eventErrors: null,
-      }));
+      .mockResolvedValue({ revision, errors: null, eventErrors: null });
 
     const result = await EventRevisionsHandler.create(ctx);
 
-    expect(mockCreate).toHaveBeenCalledWith(revision.originalId, "en");
+    expect(result).toEqual({
+      redirect: `/en/events/${event.id}`,
+    });
+  });
+
+  test("create failed when there is an opened revision for the event", async () => {
+    const event = Factory.build("event", { id: 1, revisionsCount: 2 });
+    const ctx = buildCtx(event.id);
+
+    jest.spyOn(ctx.client.events, "find").mockResolvedValue(event);
+
+    const result = await EventRevisionsHandler.create(ctx);
 
     expect(result).toEqual({
-      redirect: `/en/events/${revision.originalId}`,
+      redirect: `/en/events/${event.id}?revisions_count_error=1`,
     });
   });
 
   test("create failed with errors", async () => {
-    const ctx = buildCtx(1);
+    const event = Factory.build("event", { id: 1, revisionsCount: 0 });
     const errors = [{ key: "generalError", message: "something went wrong" }];
+    const ctx = buildCtx(event.id);
 
-    const mockCreate = jest
-      .spyOn(ctx.client.eventRevisions, "create")
-      .mockImplementation(() => ({
-        revision: null,
-        errors,
-        eventErrors: null,
-      }));
+    jest.spyOn(ctx.client.events, "find").mockResolvedValue(event);
 
-    expect(async () => {
-      await EventRevisionsHandler.create(ctx);
-    }).rejects.toThrow(`Failed to start revision: ${JSON.stringify(errors)}`);
+    jest
+      .spyOn(ctx.client.eventRevisions, "create")
+      .mockResolvedValue({ revision: null, errors, eventErrors: null });
 
-    expect(mockCreate).toHaveBeenCalledWith(1, "en");
+    await expect(EventRevisionsHandler.create(ctx)).rejects.toThrow(
+      `Failed to start revision: ${JSON.stringify(errors)}`
+    );
   });
 
   test("create failed with event errors", async () => {
-    const ctx = buildCtx(1);
+    const event = Factory.build("event", { id: 1, revisionsCount: 0 });
     const errors = [{ key: "generalError", message: "something went wrong" }];
+    const ctx = buildCtx(event.id);
 
-    const mockCreate = jest
-      .spyOn(ctx.client.eventRevisions, "create")
-      .mockImplementation(() => ({
-        revision: null,
-        errors: null,
-        eventErrors: errors,
-      }));
+    jest.spyOn(ctx.client.events, "find").mockResolvedValue(event);
 
-    expect(async () => {
-      await EventRevisionsHandler.create(ctx);
-    }).rejects.toThrow(`Failed to start revision: ${JSON.stringify(errors)}`);
+    jest
+      .spyOn(ctx.client.eventRevisions, "create")
+      .mockResolvedValue({ revision: null, errors: null, eventErrors: errors });
 
-    expect(mockCreate).toHaveBeenCalledWith(1, "en");
+    await expect(EventRevisionsHandler.create(ctx)).rejects.toThrow(
+      `Failed to start revision: ${JSON.stringify(errors)}`
+    );
   });
 });

package/src/tests/handlers/events/EventsHandler.test/show.test.js

@@ -72,7 +72,18 @@ describe("Events Handler", () => {
 
     expect(mockedeventFind).toHaveBeenCalledWith(event.id, "en");
     expect(mockedEventRevisions).toHaveBeenCalledWith(event.id, "en");
-    expect(mockedEventChanges).toHaveBeenCalledWith({}, event.id, "en");
+    expect(mockedEventChanges).toHaveBeenCalledWith(
+      {
+        "f:change_type": [
+          "revision_publish",
+          "revision_lock",
+          "revision_unlock",
+          "update",
+        ],
+      },
+      event.id,
+      "en"
+    );
 
     expect(result).toEqual({
       assigns: {

package/src/tests/handlers/page_revisions/PageRevisionsHandler.test/create.test.js

@@ -1,12 +1,13 @@
-import PageRevisionsHandler from "../../../../handlers/page_revisions/PageRevisionsHandler";
 import PageRevisions from "../../../../cloud/PageRevisions";
-
+import Pages from "../../../../cloud/Pages";
+import PageRevisionsHandler from "../../../../handlers/page_revisions/PageRevisionsHandler";
 import Factory from "../../../support/factory";
 
 function buildCtx(pageID) {
   return {
     client: {
       pageRevisions: new PageRevisions(() => ({}), "token"),
+      pages: new Pages(() => ({}), "token"),
     },
     params: { parent_id: pageID },
     locale: "en",
@@ -14,58 +15,71 @@ function buildCtx(pageID) {
       pageType: "page",
       pageTypePlural: "pages",
     },
+    maxOpenRevisions: 1,
   };
 }
 
 describe("Page revisions", () => {
   test("create successful", async () => {
+    const page = Factory.build("page", { id: 1, revisionsCount: 0 });
     const revision = Factory.build("page_revision", { originalId: 1 });
-    const ctx = buildCtx(revision.originalId);
+    const ctx = buildCtx(page.id);
+
+    jest.spyOn(ctx.client.pages, "find").mockResolvedValue(page);
 
-    const mockCreate = jest
+    jest
       .spyOn(ctx.client.pageRevisions, "create")
-      .mockImplementation(() => ({ revision, errors: null, pageErrors: null }));
+      .mockResolvedValue({ revision, errors: null, pageErrors: null });
 
     const result = await PageRevisionsHandler.create(ctx);
 
-    expect(mockCreate).toHaveBeenCalledWith(revision.originalId, "en");
+    expect(result).toEqual({
+      redirect: `/en/pages/${page.id}`,
+    });
+  });
+
+  test("create failed when there is an opened revision for the page", async () => {
+    const page = Factory.build("page", { id: 1, revisionsCount: 2 });
+    const ctx = buildCtx(page.id);
+
+    jest.spyOn(ctx.client.pages, "find").mockResolvedValue(page);
+
+    const result = await PageRevisionsHandler.create(ctx);
 
     expect(result).toEqual({
-      redirect: `/en/pages/${revision.originalId}`,
+      redirect: `/en/pages/${page.id}?revisions_count_error=1`,
     });
   });
 
   test("create failed with errors", async () => {
-    const ctx = buildCtx(1);
+    const page = Factory.build("page", { id: 1, revisionsCount: 0 });
     const errors = [{ key: "generalError", message: "something went wrong" }];
+    const ctx = buildCtx(page.id);
 
-    const mockCreate = jest
-      .spyOn(ctx.client.pageRevisions, "create")
-      .mockImplementation(() => ({ revision: null, errors, pageErrors: null }));
+    jest.spyOn(ctx.client.pages, "find").mockResolvedValue(page);
 
-    expect(async () => {
-      await PageRevisionsHandler.create(ctx);
-    }).rejects.toThrow(`Failed to start revision: ${JSON.stringify(errors)}`);
+    jest
+      .spyOn(ctx.client.pageRevisions, "create")
+      .mockResolvedValue({ revision: null, errors, pageErrors: null });
 
-    expect(mockCreate).toHaveBeenCalledWith(1, "en");
+    await expect(PageRevisionsHandler.create(ctx)).rejects.toThrow(
+      `Failed to start revision: ${JSON.stringify(errors)}`
+    );
   });
 
   test("create failed with page errors", async () => {
-    const ctx = buildCtx(1);
+    const page = Factory.build("page", { id: 1, revisionsCount: 0 });
     const errors = [{ key: "generalError", message: "something went wrong" }];
+    const ctx = buildCtx(page.id);
 
-    const mockCreate = jest
-      .spyOn(ctx.client.pageRevisions, "create")
-      .mockImplementation(() => ({
-        revision: null,
-        errors: null,
-        pageErrors: errors,
-      }));
+    jest.spyOn(ctx.client.pages, "find").mockResolvedValue(page);
 
-    expect(async () => {
-      await PageRevisionsHandler.create(ctx);
-    }).rejects.toThrow(`Failed to start revision: ${JSON.stringify(errors)}`);
+    jest
+      .spyOn(ctx.client.pageRevisions, "create")
+      .mockResolvedValue({ revision: null, errors: null, pageErrors: errors });
 
-    expect(mockCreate).toHaveBeenCalledWith(1, "en");
+    await expect(PageRevisionsHandler.create(ctx)).rejects.toThrow(
+      `Failed to start revision: ${JSON.stringify(errors)}`
+    );
   });
 });

package/src/tests/handlers/pages/PagesHandler.test/show.test.js

@@ -70,7 +70,18 @@ describe("Pages Handler", () => {
 
     expect(mockedPageFind).toHaveBeenCalledWith(page.id, "en");
     expect(mockedPageRevisions).toHaveBeenCalledWith(page.id, "en");
-    expect(mockedPageChanges).toHaveBeenCalledWith({}, page.id, "en");
+    expect(mockedPageChanges).toHaveBeenCalledWith(
+      {
+        "f:change_type": [
+          "revision_publish",
+          "revision_lock",
+          "revision_unlock",
+          "update",
+        ],
+      },
+      page.id,
+      "en"
+    );
 
     expect(result).toEqual({
       assigns: {

package/src/views/articles/_shared/_history_toolbar.ejs

@@ -2,7 +2,7 @@
   <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/${_context.resourceConfig.pageType}_changes/reverts` %>" class="inline-block">
     <input type="hidden" name="version" value="<%= change.version %>">
 
-    <% if (history.isChangeRevertable(change, maxVersion)) { %>
+    <% if (history.isChangeRevertable(change, maxVersion) && !(resource.revisionsCount >= _context.maxOpenRevisions)) { %>
     <button type="submit" class="btn-text">
       Revert
     </button>
@@ -21,4 +21,4 @@
 
     Preview
   </a>
-</div>
+</div>

package/src/views/articles/_shared/_toolbar.ejs

@@ -1,33 +1,45 @@
-<div class="actions-toolbar">
-  <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.articleTypePlural}/${resource.id}/${_context.resourceConfig.articleType}_revisions` %>" class="inline-block">
-    <button type="submit" class="btn-primary">
-      Start revision
-    </button>
-  </form>
+<div>
+  <div class="actions-toolbar">
+    <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.articleTypePlural}/${resource.id}/${_context.resourceConfig.articleType}_revisions` %>" class="inline-block">
+      <% if (resource.revisionsCount >= _context?.maxOpenRevisions) { %>
+      <button type="submit" class="btn-primary" disabled style="opacity: 50%;">
+        Start revision
+      </button>
+      <% } else {%>
+      <button type="submit" class="btn-primary">
+        Start revision
+      </button>
+      <% } %>
+    </form>
 
-  <a href="<%= `/${_context.locale}/${_context.resourceConfig.articleTypePlural}/${resource.id}/urls/new` %>" class="btn-text has-icon">
-    <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-      <path stroke-linecap="round" stroke-linejoin="round" d="M13.19 8.688a4.5 4.5 0 0 1 1.242 7.244l-4.5 4.5a4.5 4.5 0 0 1-6.364-6.364l1.757-1.757m13.35-.622 1.757-1.757a4.5 4.5 0 0 0-6.364-6.364l-4.5 4.5a4.5 4.5 0 0 0 1.242 7.244" />
-    </svg>
+    <a href="<%= `/${_context.locale}/${_context.resourceConfig.articleTypePlural}/${resource.id}/urls/new` %>" class="btn-text has-icon">
+      <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
+        <path stroke-linecap="round" stroke-linejoin="round" d="M13.19 8.688a4.5 4.5 0 0 1 1.242 7.244l-4.5 4.5a4.5 4.5 0 0 1-6.364-6.364l1.757-1.757m13.35-.622 1.757-1.757a4.5 4.5 0 0 0-6.364-6.364l-4.5 4.5a4.5 4.5 0 0 0 1.242 7.244" />
+      </svg>
+
+      Change URL
+    </a>
 
-    Change URL
-  </a>
+    <form action="<%= `/${_context.locale}/${_context.resourceConfig.articleTypePlural}/${resource.id}/clones` %>" method="post" class="inline-block">
+      <button type="submit" class="btn-text has-icon">
+        <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
+          <path stroke-linecap="round" stroke-linejoin="round" d="M15.75 17.25v3.375c0 .621-.504 1.125-1.125 1.125h-9.75a1.125 1.125 0 0 1-1.125-1.125V7.875c0-.621.504-1.125 1.125-1.125H6.75a9.06 9.06 0 0 1 1.5.124m7.5 10.376h3.375c.621 0 1.125-.504 1.125-1.125V11.25c0-4.46-3.243-8.161-7.5-8.876a9.06 9.06 0 0 0-1.5-.124H9.375c-.621 0-1.125.504-1.125 1.125v3.5m7.5 10.375H9.375a1.125 1.125 0 0 1-1.125-1.125v-9.25m12 6.625v-1.875a3.375 3.375 0 0 0-3.375-3.375h-1.5a1.125 1.125 0 0 1-1.125-1.125v-1.5a3.375 3.375 0 0 0-3.375-3.375H9.75" />
+        </svg>
 
-  <form action="<%= `/${_context.locale}/${_context.resourceConfig.articleTypePlural}/${resource.id}/clones` %>" method="post" class="inline-block">
-    <button type="submit" class="btn-text has-icon">
+        Clone <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.articleType)) %>
+      </button>
+    </form>
+
+    <a href="<%= `/${_context.locale}/${_context.resourceConfig.articleTypePlural}/${resource.id}/deletes/new` %>" class="btn-danger has-icon">
       <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-        <path stroke-linecap="round" stroke-linejoin="round" d="M15.75 17.25v3.375c0 .621-.504 1.125-1.125 1.125h-9.75a1.125 1.125 0 0 1-1.125-1.125V7.875c0-.621.504-1.125 1.125-1.125H6.75a9.06 9.06 0 0 1 1.5.124m7.5 10.376h3.375c.621 0 1.125-.504 1.125-1.125V11.25c0-4.46-3.243-8.161-7.5-8.876a9.06 9.06 0 0 0-1.5-.124H9.375c-.621 0-1.125.504-1.125 1.125v3.5m7.5 10.375H9.375a1.125 1.125 0 0 1-1.125-1.125v-9.25m12 6.625v-1.875a3.375 3.375 0 0 0-3.375-3.375h-1.5a1.125 1.125 0 0 1-1.125-1.125v-1.5a3.375 3.375 0 0 0-3.375-3.375H9.75" />
+        <path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" />
       </svg>
 
-      Clone <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.articleType)) %>
-    </button>
-  </form>
-
-  <a href="<%= `/${_context.locale}/${_context.resourceConfig.articleTypePlural}/${resource.id}/deletes/new` %>" class="btn-danger has-icon">
-    <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-      <path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" />
-    </svg>
+      Delete <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.articleType)) %>
+    </a>
+  </div>
 
-    Delete <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.articleType)) %>
-  </a>
-</div>
+  <% if (_context.query?.revisions_count_error) { %>
+  <div class="text-red-600">Cannot open a new revision. Current limit is <%= _context.maxOpenRevisions %>. </div>
+  <% } %>
+</div>

package/src/views/articles/_shared/history_table.ejs

@@ -1,3 +1,4 @@
+<% if (changes && changes.list.length > 0) { %>
 <% const maxVersion = history.getMaxVersion(changes); %>
 
 <div class="mb-4 history-table-wrapper">
@@ -66,3 +67,8 @@
   <%- pagination(changes.pagination, _context) %>
   <% } %>
 </div>
+<% } else { %>
+<div class="history-table-wrapper mb-4">
+  <p class="text-center py-4">No changes, yet.</p>
+</div>
+<% } %>

package/src/views/events/_shared/_history_toolbar.ejs

@@ -2,7 +2,7 @@
   <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/${_context.resourceConfig.pageType}_changes/reverts` %>" class="inline-block">
     <input type="hidden" name="version" value="<%= change.version %>">
 
-    <% if (history.isChangeRevertable(change, maxVersion)) { %>
+    <% if (history.isChangeRevertable(change, maxVersion) && !(resource.revisionsCount >= _context.maxOpenRevisions)) { %>
     <button type="submit" class="btn-text">
       Revert
     </button>
@@ -21,4 +21,4 @@
 
     Preview
   </a>
-</div>
+</div>

package/src/views/events/_shared/_toolbar.ejs

@@ -1,33 +1,45 @@
-<div class="actions-toolbar">
-  <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.eventTypePlural}/${resource.id}/${_context.resourceConfig.eventType}_revisions` %>" class="inline-block">
-    <button type="submit" class="btn-primary">
-      Start revision
-    </button>
-  </form>
+<div>
+  <div class="actions-toolbar">
+    <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.eventTypePlural}/${resource.id}/${_context.resourceConfig.eventType}_revisions` %>" class="inline-block">
+      <% if (resource.revisionsCount >= _context?.maxOpenRevisions) { %>
+      <button type="submit" class="btn-primary" disabled style="opacity: 50%;">
+        Start revision
+      </button>
+      <% } else {%>
+      <button type="submit" class="btn-primary">
+        Start revision
+      </button>
+      <% } %>
+    </form>
 
-  <a href="<%= `/${_context.locale}/${_context.resourceConfig.eventTypePlural}/${resource.id}/urls/new` %>" class="btn-text has-icon">
-    <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-      <path stroke-linecap="round" stroke-linejoin="round" d="M13.19 8.688a4.5 4.5 0 0 1 1.242 7.244l-4.5 4.5a4.5 4.5 0 0 1-6.364-6.364l1.757-1.757m13.35-.622 1.757-1.757a4.5 4.5 0 0 0-6.364-6.364l-4.5 4.5a4.5 4.5 0 0 0 1.242 7.244" />
-    </svg>
+    <a href="<%= `/${_context.locale}/${_context.resourceConfig.eventTypePlural}/${resource.id}/urls/new` %>" class="btn-text has-icon">
+      <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
+        <path stroke-linecap="round" stroke-linejoin="round" d="M13.19 8.688a4.5 4.5 0 0 1 1.242 7.244l-4.5 4.5a4.5 4.5 0 0 1-6.364-6.364l1.757-1.757m13.35-.622 1.757-1.757a4.5 4.5 0 0 0-6.364-6.364l-4.5 4.5a4.5 4.5 0 0 0 1.242 7.244" />
+      </svg>
+
+      Change URL
+    </a>
 
-    Change URL
-  </a>
+    <form action="<%= `/${_context.locale}/${_context.resourceConfig.eventTypePlural}/${resource.id}/clones` %>" method="post" class="inline-block">
+      <button type="submit" class="btn-text has-icon">
+        <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
+          <path stroke-linecap="round" stroke-linejoin="round" d="M15.75 17.25v3.375c0 .621-.504 1.125-1.125 1.125h-9.75a1.125 1.125 0 0 1-1.125-1.125V7.875c0-.621.504-1.125 1.125-1.125H6.75a9.06 9.06 0 0 1 1.5.124m7.5 10.376h3.375c.621 0 1.125-.504 1.125-1.125V11.25c0-4.46-3.243-8.161-7.5-8.876a9.06 9.06 0 0 0-1.5-.124H9.375c-.621 0-1.125.504-1.125 1.125v3.5m7.5 10.375H9.375a1.125 1.125 0 0 1-1.125-1.125v-9.25m12 6.625v-1.875a3.375 3.375 0 0 0-3.375-3.375h-1.5a1.125 1.125 0 0 1-1.125-1.125v-1.5a3.375 3.375 0 0 0-3.375-3.375H9.75" />
+        </svg>
 
-  <form action="<%= `/${_context.locale}/${_context.resourceConfig.eventTypePlural}/${resource.id}/clones` %>" method="post" class="inline-block">
-    <button type="submit" class="btn-text has-icon">
+        Clone <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.eventType)) %>
+      </button>
+    </form>
+
+    <a href="<%= `/${_context.locale}/${_context.resourceConfig.eventTypePlural}/${resource.id}/deletes/new` %>" class="btn-danger has-icon">
       <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-        <path stroke-linecap="round" stroke-linejoin="round" d="M15.75 17.25v3.375c0 .621-.504 1.125-1.125 1.125h-9.75a1.125 1.125 0 0 1-1.125-1.125V7.875c0-.621.504-1.125 1.125-1.125H6.75a9.06 9.06 0 0 1 1.5.124m7.5 10.376h3.375c.621 0 1.125-.504 1.125-1.125V11.25c0-4.46-3.243-8.161-7.5-8.876a9.06 9.06 0 0 0-1.5-.124H9.375c-.621 0-1.125.504-1.125 1.125v3.5m7.5 10.375H9.375a1.125 1.125 0 0 1-1.125-1.125v-9.25m12 6.625v-1.875a3.375 3.375 0 0 0-3.375-3.375h-1.5a1.125 1.125 0 0 1-1.125-1.125v-1.5a3.375 3.375 0 0 0-3.375-3.375H9.75" />
+        <path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" />
       </svg>
 
-      Clone <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.eventType)) %>
-    </button>
-  </form>
-
-  <a href="<%= `/${_context.locale}/${_context.resourceConfig.eventTypePlural}/${resource.id}/deletes/new` %>" class="btn-danger has-icon">
-    <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-      <path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" />
-    </svg>
+      Delete <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.eventType)) %>
+    </a>
+  </div>
 
-    Delete <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.eventType)) %>
-  </a>
-</div>
+  <% if (_context.query?.revisions_count_error) { %>
+  <div class="text-red-600">Cannot open a new revision. Current limit is <%= _context.maxOpenRevisions %>. </div>
+  <% } %>
+</div>

package/src/views/events/_shared/history_table.ejs

@@ -1,3 +1,4 @@
+<% if (changes && changes.list.length > 0) { %>
 <% const maxVersion = history.getMaxVersion(changes); %>
 
 <div class="history-table-wrapper mb-4">
@@ -66,3 +67,8 @@
   <%- pagination(changes.pagination, _context) %>
   <% } %>
 </div>
+<% } else { %>
+<div class="history-table-wrapper mb-4">
+  <p class="text-center py-4">No changes, yet.</p>
+</div>
+<% } %>

package/src/views/pages/_shared/_history_toolbar.ejs

@@ -2,7 +2,7 @@
   <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/${_context.resourceConfig.pageType}_changes/reverts` %>" class="inline-block">
     <input type="hidden" name="version" value="<%= change.version %>">
 
-    <% if (history.isChangeRevertable(change, maxVersion)) { %>
+    <% if (history.isChangeRevertable(change, maxVersion) && !(resource.revisionsCount >= _context.maxOpenRevisions)) { %>
     <button type="submit" class="btn-text">
       Revert
     </button>
@@ -21,4 +21,4 @@
 
     Preview
   </a>
-</div>
+</div>

package/src/views/pages/_shared/_toolbar.ejs

@@ -1,33 +1,45 @@
-<div class="actions-toolbar">
-  <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/${_context.resourceConfig.pageType}_revisions` %>" class="inline-block">
-    <button type="submit" class="btn-primary">
-      Start revision
-    </button>
-  </form>
+<div>
+  <div class="actions-toolbar">
+    <form method="POST" action="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/${_context.resourceConfig.pageType}_revisions` %>" class="inline-block">
+      <% if (resource.revisionsCount >= _context?.maxOpenRevisions) { %>
+      <button type="submit" class="btn-primary" disabled style="opacity: 50%;">
+        Start revision
+      </button>
+      <% } else {%>
+      <button type="submit" class="btn-primary">
+        Start revision
+      </button>
+      <% } %>
+    </form>
 
-  <a href="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/urls/new` %>" class="btn-text has-icon">
-    <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-      <path stroke-linecap="round" stroke-linejoin="round" d="M13.19 8.688a4.5 4.5 0 0 1 1.242 7.244l-4.5 4.5a4.5 4.5 0 0 1-6.364-6.364l1.757-1.757m13.35-.622 1.757-1.757a4.5 4.5 0 0 0-6.364-6.364l-4.5 4.5a4.5 4.5 0 0 0 1.242 7.244" />
-    </svg>
+    <a href="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/urls/new` %>" class="btn-text has-icon">
+      <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
+        <path stroke-linecap="round" stroke-linejoin="round" d="M13.19 8.688a4.5 4.5 0 0 1 1.242 7.244l-4.5 4.5a4.5 4.5 0 0 1-6.364-6.364l1.757-1.757m13.35-.622 1.757-1.757a4.5 4.5 0 0 0-6.364-6.364l-4.5 4.5a4.5 4.5 0 0 0 1.242 7.244" />
+      </svg>
+
+      Change URL
+    </a>
 
-    Change URL
-  </a>
+    <form action="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/clones` %>" method="post" class="inline-block">
+      <button type="submit" class="btn-text has-icon">
+        <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
+          <path stroke-linecap="round" stroke-linejoin="round" d="M15.75 17.25v3.375c0 .621-.504 1.125-1.125 1.125h-9.75a1.125 1.125 0 0 1-1.125-1.125V7.875c0-.621.504-1.125 1.125-1.125H6.75a9.06 9.06 0 0 1 1.5.124m7.5 10.376h3.375c.621 0 1.125-.504 1.125-1.125V11.25c0-4.46-3.243-8.161-7.5-8.876a9.06 9.06 0 0 0-1.5-.124H9.375c-.621 0-1.125.504-1.125 1.125v3.5m7.5 10.375H9.375a1.125 1.125 0 0 1-1.125-1.125v-9.25m12 6.625v-1.875a3.375 3.375 0 0 0-3.375-3.375h-1.5a1.125 1.125 0 0 1-1.125-1.125v-1.5a3.375 3.375 0 0 0-3.375-3.375H9.75" />
+        </svg>
 
-  <form action="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/clones` %>" method="post" class="inline-block">
-    <button type="submit" class="btn-text has-icon">
+        Clone <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.pageType)) %>
+      </button>
+    </form>
+
+    <a href="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/deletes/new` %>" class="btn-danger has-icon">
       <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-        <path stroke-linecap="round" stroke-linejoin="round" d="M15.75 17.25v3.375c0 .621-.504 1.125-1.125 1.125h-9.75a1.125 1.125 0 0 1-1.125-1.125V7.875c0-.621.504-1.125 1.125-1.125H6.75a9.06 9.06 0 0 1 1.5.124m7.5 10.376h3.375c.621 0 1.125-.504 1.125-1.125V11.25c0-4.46-3.243-8.161-7.5-8.876a9.06 9.06 0 0 0-1.5-.124H9.375c-.621 0-1.125.504-1.125 1.125v3.5m7.5 10.375H9.375a1.125 1.125 0 0 1-1.125-1.125v-9.25m12 6.625v-1.875a3.375 3.375 0 0 0-3.375-3.375h-1.5a1.125 1.125 0 0 1-1.125-1.125v-1.5a3.375 3.375 0 0 0-3.375-3.375H9.75" />
+        <path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" />
       </svg>
 
-      Clone <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.pageType)) %>
-    </button>
-  </form>
-
-  <a href="<%= `/${_context.locale}/${_context.resourceConfig.pageTypePlural}/${resource.id}/deletes/new` %>" class="btn-danger has-icon">
-    <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4">
-      <path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" />
-    </svg>
+      Delete <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.pageType)) %>
+    </a>
+  </div>
 
-    Delete <%= Inflector.humanize(Inflector.downcase(_context.resourceConfig.pageType)) %>
-  </a>
-</div>
+  <% if (_context.query?.revisions_count_error) { %>
+  <div class="text-red-600">Cannot open a new revision. Current limit is <%= _context.maxOpenRevisions %>. </div>
+  <% } %>
+</div>

package/src/views/pages/_shared/history_table.ejs

@@ -1,3 +1,4 @@
+<% if (changes && changes.list.length > 0) { %>
 <% const maxVersion = history.getMaxVersion(changes); %>
 
 <div class="history-table-wrapper mb-4">
@@ -66,3 +67,8 @@
   <%- pagination(changes.pagination, _context) %>
   <% } %>
 </div>
+<% } else { %>
+<div class="history-table-wrapper mb-4">
+  <p class="text-center py-4">No changes, yet.</p>
+</div>
+<% } %>