Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[0.12.x] Fix missing filters for get settings validator (#5480) #5522

Merged
merged 3 commits into from Sep 11, 2023

Conversation

wolflu05
Copy link
Contributor

Backport

This will backport the following commits from master to 0.12.x:

Questions ?

Please refer to the Backport tool documentation

* Fix missing filters for get settings validator

* merge default model instance filters and kwargs

* Added tests for validators

* Give it a try without the kwargs passed to clean in save function

* Added string for identification for debug statement

* Added more debug comments

* Added more debug prints

* Fix test debug

* Modiefied workflow

* trigger ci

* Fix test and remove unused kwargs

* Added debug prints

* Only run one test in ci

* Added more debug code

* Remove all debug prints and reset workflow

* Reset overlooked file

(cherry picked from commit 9a6c2d2)

# Conflicts:
#	InvenTree/plugin/samples/integration/test_sample.py
@SchrodingersGat SchrodingersGat added this to the 0.12.7 milestone Sep 10, 2023
@SchrodingersGat SchrodingersGat added the bug Identifies a bug which needs to be addressed label Sep 10, 2023
@wolflu05
Copy link
Contributor Author

wolflu05 commented Sep 10, 2023

Don't wonder, I tried out this back port tool locally by adding a PAT because I thought it will be easier, but no, actually it was some kind of weird first. Not sure if it wouldn't be easier to do it with git directly if I need to manually do it again.

@wolflu05
Copy link
Contributor Author

Should work now, so it can be merged. I think the rest of the failures are not related to my changes, because they are also present in the previous runs on the 0.12.x branch.

@SchrodingersGat SchrodingersGat merged commit 09872ee into inventree:0.12.x Sep 11, 2023
12 of 14 checks passed
@wolflu05 wolflu05 deleted the backport/0.12.x/pr-5480 branch February 15, 2024 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Identifies a bug which needs to be addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants