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

Improve UpdateConfig and ConfigurationValidator #947

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

M0rgan01
Copy link
Contributor

@M0rgan01 M0rgan01 commented Oct 9, 2024

Questions Answers
Description? Refactoring of the UpdateConfig task, adding cases in the ConfigurationValidator.
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company -
How to test? Taking the configuration into account works as before, on the CLI and on the WEB version

@M0rgan01 M0rgan01 added this to the 7.0.0 milestone Oct 9, 2024
@M0rgan01 M0rgan01 changed the title Improve config validator Improve UpdateConfig and ConfigurationValidator Oct 9, 2024
@M0rgan01 M0rgan01 marked this pull request as ready for review October 10, 2024 08:26
@M0rgan01 M0rgan01 closed this Oct 10, 2024
@M0rgan01 M0rgan01 reopened this Oct 10, 2024
nicosomb
nicosomb previously approved these changes Oct 10, 2024
@M0rgan01 M0rgan01 force-pushed the improve-config-validator branch 3 times, most recently from 652599b to c42aea8 Compare October 10, 2024 13:33
nicosomb
nicosomb previously approved these changes Oct 10, 2024
ga-devfront
ga-devfront previously approved these changes Oct 14, 2024
Quetzacoalt91
Quetzacoalt91 previously approved these changes Oct 15, 2024
@Quetzacoalt91
Copy link
Member

Quetzacoalt91 commented Oct 15, 2024

Even though this PR is fixing an issue, there is still an error happening while reaching the configuration page.
Worth making the third method as nullable?

image

Copy link

sonarcloud bot commented Oct 15, 2024

@Quetzacoalt91 Quetzacoalt91 merged commit cb16318 into PrestaShop:dev Oct 15, 2024
36 checks passed
@Quetzacoalt91 Quetzacoalt91 deleted the improve-config-validator branch October 15, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Reopened
Development

Successfully merging this pull request may close these issues.

5 participants