Skip to content

Change log level for missing field during model binding #31

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

Merged
merged 1 commit into from
Apr 7, 2025

Conversation

jballe
Copy link
Contributor

@jballe jballe commented Mar 11, 2025

This fixes issue #30

Description / Motivation

Changes level of log level during model binding when field is missing from warning to debug

Testing

  • The Unit & Intergration tests are passing.
  • I have added the necessary tests to cover my changes.

Terms

@jballe
Copy link
Contributor Author

jballe commented Mar 11, 2025

I have to change regional settings for all (date related) tests to pass - but that is obviously not related to this change :)
I have created issue #34 about that.

@robearlam robearlam linked an issue Mar 11, 2025 that may be closed by this pull request
@sc-ivanlieckens sc-ivanlieckens self-assigned this Apr 7, 2025
@sc-ivanlieckens sc-ivanlieckens added the enhancement New feature or request label Apr 7, 2025
@sc-ivanlieckens sc-ivanlieckens moved this to In review in ASP.NET Core SDK Apr 7, 2025
@sc-ivanlieckens sc-ivanlieckens merged commit 0abe9f5 into Sitecore:main Apr 7, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In review to Done in ASP.NET Core SDK Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Warnings during model binding
2 participants