Skip to content

Conversation

@atownson
Copy link
Contributor

Fixes: #18369

Removing the json filter resolves the TypeError exception and aligns the protection rules template with the custom validators template.

Copy link
Member

@jeremystretch jeremystretch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @atownson!

@jeremystretch jeremystretch merged commit 2ed4a2b into netbox-community:main Jan 17, 2025
3 checks passed
bctiemann pushed a commit that referenced this pull request Jan 28, 2025
* Remove the json filter for protection rules

* Configure PROTECTION_RULE config attribute to use ConfigJSONEncoder as serializer

* Tweak getattr()

---------

Co-authored-by: Jeremy Stretch <[email protected]>
bctiemann pushed a commit that referenced this pull request Jan 29, 2025
* Remove the json filter for protection rules

* Configure PROTECTION_RULE config attribute to use ConfigJSONEncoder as serializer

* Tweak getattr()

---------

Co-authored-by: Jeremy Stretch <[email protected]>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 17, 2025
@atownson atownson deleted the issue_18369 branch February 24, 2025 15:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JSON serializer issue for Protection Rules defined in configuration.py

2 participants