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

Bug fix in "backend.config.Settings" #6587

Closed
wants to merge 1 commit into from

Conversation

Rojf
Copy link

@Rojf Rojf commented Oct 2, 2024

What type of PR is this? (check all applicable)

  • 🐛 Bug Fix

Describe this PR

There are two problems here:

pedantic_core._pedantic_core.Validation Error: 1 validation error for Settings SENTRY_BACKEND_DSN
It was necessary to add Optional[str]

pedantic_or._pedantic_core.Validation Error: 3 validation errors for Connection Config MAIL_USERNAME
MAIL_PASSWORD
MAIL_SERVER
It should have been changed from None to ""
The "ConnectionConfig" class is not "Optional[str]" for pydantic

Bug fix in "backend.config.Settings"

There are two problems here:

pedantic_core._pedantic_core.Validation Error: 1 validation error for Settings
SENTRY_BACKEND_DSN
It was necessary to add Optional[str]

pedantic_or._pedantic_core.Validation Error: 3 validation errors for Connection Config
MAIL_USERNAME
MAIL_PASSWORD
MAIL_SERVER
It should have been changed from None to ""
The "ConnectionConfig" class is not optional for pydantic
Copy link

sonarcloud bot commented Oct 2, 2024

@Rojf Rojf closed this Oct 3, 2024
@Rojf Rojf deleted the patch-1 branch October 3, 2024 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant