[PR #120] [MERGED] Remove `None` values from request params and data (Fixes #119) #183

New issue

Closed

opened 2026-02-27 15:46:48 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 15:46:48 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/proxmoxer/proxmoxer/pull/120
Author: @kristianheljas
Created: 11/19/2022
Status: ✅ Merged
Merged: 11/20/2022
Merged by: @jhollowe

Base: develop ← Head: remove-none-params

📝 Commits (1)

f1bf6bf Remove None values from request data and params

📊 Changes

2 files changed (+26 additions, -1 deletions)

View changed files

📝 proxmoxer/core.py (+16 -1)
📝 tests/test_core.py (+10 -0)

📄 Description

In addition to cleaning up params and data, as discussed in #119 fixed data integrity checks as well.
I can separate the data integrity check fix to a separate PR if needed.

_{🔄 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/proxmoxer/proxmoxer/pull/120 **Author:** [@kristianheljas](https://github.com/kristianheljas) **Created:** 11/19/2022 **Status:** ✅ Merged **Merged:** 11/20/2022 **Merged by:** [@jhollowe](https://github.com/jhollowe) **Base:** `develop` ← **Head:** `remove-none-params` --- ### 📝 Commits (1) - [`f1bf6bf`](https://github.com/proxmoxer/proxmoxer/commit/f1bf6bf6420e08455068f9744d48c011ce7514d2) Remove `None` values from request data and params ### 📊 Changes **2 files changed** (+26 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `proxmoxer/core.py` (+16 -1) 📝 `tests/test_core.py` (+10 -0) </details> ### 📄 Description In addition to cleaning up params and data, as discussed in #119 fixed data integrity checks as well. I can separate the data integrity check fix to a separate PR if needed. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>