Bug Report Deion links in index.html doe

How do you enable that? <a class="user-mention notranslate" data-hov

You can set it in next.config.js: <a href="https://nextjs.org/docs/api-reference/next.

links in index.html does not contain `.html` suffix, but generated html files has about log4brains HOT 5 CLOSED

thomvaill commented on May 27, 2024

links in index.html does not contain `.html` suffix, but generated html files has

from log4brains.

Comments (5)

deviantintegral commented on May 27, 2024

At a basic level, setting trailingSlash: true, fixes the issue for me. As noted in the stack overflow threads, it does mean ADRs are rendered as directories, such as http://localhost/adr/0001-record-architecture-decisions/. However, that could actually be a good thing if we get inline images working, because then those assets could be included in the ADR path.

from log4brains.

fire commented on May 27, 2024

How do you enable that?

@deviantintegral

from log4brains.

deviantintegral commented on May 27, 2024

You can set it in next.config.js: https://nextjs.org/docs/api-reference/next.config.js/trailing-slash

from log4brains.

llwt commented on May 27, 2024

For anyone coming to this issue looking to apply this fix, you can use patch-package while we wait for this change to be merged:

The autogenerated output from patch-package is below :)

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch @log4brains/[email protected] for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/@log4brains/web/dist/next.config.js b/node_modules/@log4brains/web/dist/next.config.js
index a5dd2e9..4d14fab 100644
--- a/node_modules/@log4brains/web/dist/next.config.js
+++ b/node_modules/@log4brains/web/dist/next.config.js
@@ -13,6 +13,16 @@ module.exports = withBundleAnalyzer({
   reactStrictMode: true,
   target: "serverless",
   poweredByHeader: false,
+  /**
+   * This patch is to ensure that next outputs page/index.html instead of page.html
+   * Without this, links to pages other than the index file will not work on s3
+   * 
+   * without:
+   *   .log4brains/out/adr/2022022-document-frontend-infra-decisions.html
+   * with:
+   *   .log4brains/out/adr/2022022-document-frontend-infra-decisions/index.html
+   */
+  trailingSlash: true,
   serverRuntimeConfig: {
     PROJECT_ROOT: __dirname, // https://github.com/vercel/next.js/issues/8251
     VERSION: process.env.HIDE_LOG4BRAINS_VERSION ? "" : packageJson.version