Sync changes in ApplicationSetting to Default Organization
1 unresolved thread
1 unresolved thread
Compare changes
Files
4+ 35
− 0
In MR !123380 (merged), we added one ApplicationSetting (restricted_visibility_levels
) to OrganizationSetting
model.
This MR will sync changes in ApplicationSetting
to OrganizationSetting
. This is needed because we do not have a UI for OrganizationSettings.
ApplicationSettings::SyncService
ApplicationSettings::UpdaterService
which is used in a few places, including the controller that handles updates from the UIWe have a few database migrations that are modifying ApplicationSetting
without using this UpdaterService
or the ActiveRecord model. Those changes are missed in this MR. At some point, we will start to remove settings from ApplicationSetting so I think it is acceptable. New settings need to be added either to ApplicationSetting (instance level) or OrganzationSetting (organization level).
We could also consider adding a job that runs this service periodically.
ApplicationSetting.first.restriced_visiblity_levels
should be updatedOrganizations::Organization.default_organization.settings.restricted_visibility_levels
should be updated and match ApplicationSettingThis checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #394801 (closed)