[PR #109] [MERGED] website: manually set basePath in next.config.mjs #2398

Closed
opened 2025-11-20 05:29:39 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/109
Author: @BramSuurdje
Created: 11/6/2024
Status: Merged
Merged: 11/6/2024
Merged by: @BramSuurdje

Base: mainHead: main


📝 Commits (1)

  • ada1a53 manually set basePath in next.config.mjs

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 frontend/next.config.mjs (+1 -1)

📄 Description

Note

We are meticulous when it comes to merging code into the main branch, so please understand that we may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged.

Description

manually set basePath in next.config.mjs to avoid errors while building. while testing locally with bun run build and then starting i had no issues with having it as an env var.

Fixes # (issue)

Type of change

Please check the relevant option(s):

  • Bug fix (non-breaking change that resolves an issue)

Prerequisites

The following efforts must be made for the PR to be considered. Please check when completed:

  • Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions)
  • Testing performed (I have tested my changes, ensuring everything works as expected)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/community-scripts/ProxmoxVE/pull/109 **Author:** [@BramSuurdje](https://github.com/BramSuurdje) **Created:** 11/6/2024 **Status:** ✅ Merged **Merged:** 11/6/2024 **Merged by:** [@BramSuurdje](https://github.com/BramSuurdje) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (1) - [`ada1a53`](https://github.com/community-scripts/ProxmoxVE/commit/ada1a537e5ad135766835f99b0803c9bd9ec3701) manually set basePath in `next.config.mjs` ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `frontend/next.config.mjs` (+1 -1) </details> ### 📄 Description > [!NOTE] > We are meticulous when it comes to merging code into the main branch, so please understand that we may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged. ## Description manually set basePath in `next.config.mjs` to avoid errors while building. while testing locally with `bun run build` and then starting i had no issues with having it as an env var. Fixes # (issue) ## Type of change Please check the relevant option(s): - [x] Bug fix (non-breaking change that resolves an issue) ## Prerequisites The following efforts must be made for the PR to be considered. Please check when completed: - [x] Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions) - [x] Testing performed (I have tested my changes, ensuring everything works as expected) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2025-11-20 05:29:39 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#2398