starred/netbox
1
0
Fork 0
mirror of https://github.com/netbox-community/netbox.git synced 2026-01-11 21:10:29 +01:00
Code Issues 273 Packages Projects Releases 10 Wiki Activity

[PR #7172] [CLOSED] Fixes #7162: Fix API-populated form fields when BASE_PATH is defined #13217

New Issue
Closed
opened 2025-12-29 22:26:03 +01:00 by adam · 0 comments
adam commented 2025-12-29 22:26:03 +01:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/7172
Author: @jeremystretch
Created: 9/3/2021
Status: ❌ Closed

Base: develop ← Head: 7162-base-path-bug

📝 Commits (3)

  • acb2448 Fixes #7162: Fix API-populated form fields when BASE_PATH is defined
  • 70cd096 Fixes #7162: Ensure BASE_PATH is only included in API calls once
  • c35563d #7162: Don't modify the original API URL when prepending BASE_PATH

📊 Changes

12 files changed (+26 additions, -14 deletions)

View changed files

📝 docs/release-notes/version-3.0.md (+1 -0)
📝 netbox/project-static/dist/config.js (+1 -1)
📝 netbox/project-static/dist/config.js.map (+1 -1)
📝 netbox/project-static/dist/jobs.js (+1 -1)
📝 netbox/project-static/dist/jobs.js.map (+1 -1)
📝 netbox/project-static/dist/lldp.js (+1 -1)
📝 netbox/project-static/dist/lldp.js.map (+1 -1)
📝 netbox/project-static/dist/netbox.js (+1 -1)
📝 netbox/project-static/dist/netbox.js.map (+1 -1)
📝 netbox/project-static/dist/status.js (+2 -2)
📝 netbox/project-static/dist/status.js.map (+1 -1)
📝 netbox/project-static/src/util.ts (+14 -3)

📄 Description

Fixes: #7162

  • Remove the call to buildUrl() from apiRequest
  • Pass the table config API endpoint through buildUrl() within submitFormConfig

🔄 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/netbox-community/netbox/pull/7172 **Author:** [@jeremystretch](https://github.com/jeremystretch) **Created:** 9/3/2021 **Status:** ❌ Closed **Base:** `develop` ← **Head:** `7162-base-path-bug` --- ### 📝 Commits (3) - [`acb2448`](https://github.com/netbox-community/netbox/commit/acb24489d1f94e0320f4d099d70f1c2390a36c9f) Fixes #7162: Fix API-populated form fields when BASE_PATH is defined - [`70cd096`](https://github.com/netbox-community/netbox/commit/70cd0969e423159d3b71db6897445f7180e76b8f) Fixes #7162: Ensure `BASE_PATH` is only included in API calls once - [`c35563d`](https://github.com/netbox-community/netbox/commit/c35563d82f047f23f4b616cd75f065df2fd9b2ad) #7162: Don't modify the original API URL when prepending `BASE_PATH` ### 📊 Changes **12 files changed** (+26 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `docs/release-notes/version-3.0.md` (+1 -0) 📝 `netbox/project-static/dist/config.js` (+1 -1) 📝 `netbox/project-static/dist/config.js.map` (+1 -1) 📝 `netbox/project-static/dist/jobs.js` (+1 -1) 📝 `netbox/project-static/dist/jobs.js.map` (+1 -1) 📝 `netbox/project-static/dist/lldp.js` (+1 -1) 📝 `netbox/project-static/dist/lldp.js.map` (+1 -1) 📝 `netbox/project-static/dist/netbox.js` (+1 -1) 📝 `netbox/project-static/dist/netbox.js.map` (+1 -1) 📝 `netbox/project-static/dist/status.js` (+2 -2) 📝 `netbox/project-static/dist/status.js.map` (+1 -1) 📝 `netbox/project-static/src/util.ts` (+14 -3) </details> ### 📄 Description ### Fixes: #7162 - Remove the call to `buildUrl()` from `apiRequest` - Pass the table config API endpoint through `buildUrl()` within `submitFormConfig` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 22:26:03 +01:00
adam closed this issue 2025-12-29 22:26:03 +01:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
beta
breaking change
complexity: high
complexity: low
complexity: medium
needs milestone
netbox
pending closure
plugin candidate
pull-request
Mirrored from GitHub Pull Request
 severity: high
severity: low
severity: medium
status: accepted
status: backlog
status: blocked
status: duplicate
status: needs owner
status: needs triage
status: revisions needed
status: under review
topic: GraphQL
topic: Internationalization
topic: OpenAPI
topic: UI/UX
topic: cabling
topic: event rules
topic: htmx navigation
topic: industrialization
topic: migrations
topic: plugins
topic: scripts
topic: templating
topic: testing
type: bug
type: deprecation
type: documentation
type: feature
type: housekeeping
type: translation
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adam
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13217
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?