Content
View differences
Updated by Christophe Bliard about 4 years ago
> Don't think that it will cause an immediate problem but since the purpose of this PR is to clean up I am going to be a bit pedantic.
>
> If the seeding ran before the fix, the database will still contain the empty strings for e.g.
> `Setting.oauth_allow_remapping_of_existing_users`
>
> It will now result in `nil` rather than `''` but that is different from the expected behaviour.
[https://github.com/opf/openproject/pull/10508](https://github.com/opf/openproject/pull/10508)
[https://github.com/opf/openproject/pull/10574](https://github.com/opf/openproject/pull/10574) https://github.com/opf/openproject/pull/10508
>
> If the seeding ran before the fix, the database will still contain the empty strings for e.g.
> `Setting.oauth_allow_remapping_of_existing_users`
>
> It will now result in `nil` rather than `''` but that is different from the expected behaviour.
[https://github.com/opf/openproject/pull/10508](https://github.com/opf/openproject/pull/10508)
[https://github.com/opf/openproject/pull/10574](https://github.com/opf/openproject/pull/10574)