[PR #12527] [MERGED] Adds full_clean in examples #14003

Closed
opened 2025-12-29 23:21:59 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/12527
Author: @abhi1693
Created: 5/8/2023
Status: Merged
Merged: 5/9/2023
Merged by: @jeremystretch

Base: developHead: feat/11689-script-doc


📝 Commits (2)

📊 Changes

2 files changed (+4 additions, -6 deletions)

View changed files

📝 docs/administration/netbox-shell.md (+2 -6)
📝 docs/customization/custom-scripts.md (+2 -0)

📄 Description

Fixes: #11689

One question, since it's not possible to provide a 1-liner along with full_clean, should I remove this from the doc?

https://github.com/netbox-community/netbox/blob/develop/docs/administration/netbox-shell.md?plain=1#L159-L163

Alternatively, the above can be performed as a single operation. (Note, however, that `save()` does _not_ return the new instance for reuse.)

>>> VLAN(vid=123, name='MyNewVLAN', site=Site.objects.get(pk=7)).save()

🔄 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/12527 **Author:** [@abhi1693](https://github.com/abhi1693) **Created:** 5/8/2023 **Status:** ✅ Merged **Merged:** 5/9/2023 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `develop` ← **Head:** `feat/11689-script-doc` --- ### 📝 Commits (2) - [`3f56f13`](https://github.com/netbox-community/netbox/commit/3f56f13d84bc7954d5e1955fea36cd2fa3799c75) adds full_clean in examples #11689 - [`9c728bf`](https://github.com/netbox-community/netbox/commit/9c728bf58e41943259fc6d775733642a1df88431) removes extra info ### 📊 Changes **2 files changed** (+4 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `docs/administration/netbox-shell.md` (+2 -6) 📝 `docs/customization/custom-scripts.md` (+2 -0) </details> ### 📄 Description <!-- Thank you for your interest in contributing to NetBox! Please note that our contribution policy requires that a feature request or bug report be approved and assigned prior to opening a pull request. This helps avoid waste time and effort on a proposed change that we might not be able to accept. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ISSUE WHICH HAS BEEN ASSIGNED TO YOU, IT WILL BE CLOSED AUTOMATICALLY. Please specify your assigned issue number on the line below. --> ### Fixes: #11689 <!-- Please include a summary of the proposed changes below. --> One question, since it's not possible to provide a 1-liner along with `full_clean`, should I remove this from the doc? https://github.com/netbox-community/netbox/blob/develop/docs/administration/netbox-shell.md?plain=1#L159-L163 ``` Alternatively, the above can be performed as a single operation. (Note, however, that `save()` does _not_ return the new instance for reuse.) >>> VLAN(vid=123, name='MyNewVLAN', site=Site.objects.get(pk=7)).save() ``` --- <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 23:21:59 +01:00
adam closed this issue 2025-12-29 23:21:59 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#14003