Project

General

Profile

Tracker #29939

Move setting registry into a memory

Added by Ondřej Ezr 8 months ago. Updated about 1 month ago.

Status:
Ready For Testing
Priority:
Normal
Assignee:
Ondřej Ezr
Category:
Settings
Target version:
2.4.0
% Done:

50%

Difficulty:
medium
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Currently we have all the settings (both changed and unchanged) in the database, but we have all the defaults in code as well, so it makes no sense to have quite a heavy code base to keep both in sync.
Instead I'd like to bring the setting inventory into memory and database would keep only the changed settings.

It should give as an easy way for plugins to extend settings category not just adding a new one.

Subtasks

Refactor #30860: Create SettingPresenter as a proxy for setting UI valuesClosedOndřej Ezr
Refactor #30861: Move setting value collections to SettingPresenterNewOndřej Ezr
Feature #30862: Introduce SettingRegistry as a setting inventoryReady For TestingOndřej Ezr
Refactor #31471: Deprecate setting timestamps in API responsesClosedOndřej Ezr

Related issues

Related to Katello - Refactor #30354: Use only public setting interface to update setting valuesClosed
Related to Foreman - Refactor #29257: remove x-editable-railsClosed

History

#1 Updated by The Foreman Bot 8 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7702 added

#2 Updated by Ondřej Ezr 7 months ago

  • Related to Refactor #30354: Use only public setting interface to update setting values added

#3 Updated by Amit Upadhye 5 months ago

  • Target version changed from 2.2.0 to 2.3.0

#4 Updated by Ondřej Ezr 4 months ago

  • Subject changed from Add extension point to setting defaults to Move setting registry into a memory

#5 Updated by Ondřej Ezr 4 months ago

  • Description updated (diff)

#6 Updated by Ondřej Ezr 4 months ago

  • % Done set to 0
  • Tracker changed from Feature to Tracker
  • Pull request deleted (https://github.com/theforeman/foreman/pull/7702)

#7 Updated by Ondřej Ezr 4 months ago

  • Description updated (diff)

#8 Updated by Ondřej Ezr 4 months ago

#9 Updated by Ondřej Ezr 2 months ago

  • Target version changed from 2.3.0 to 2.4.0

Also available in: Atom PDF