Skip to content

feat(admin): adding settings page#1818

Merged
netomi merged 1 commit into
eclipse-openvsx:read-only-modefrom
gnugomez:gnugomez/master/settings-admin-page
May 12, 2026
Merged

feat(admin): adding settings page#1818
netomi merged 1 commit into
eclipse-openvsx:read-only-modefrom
gnugomez:gnugomez/master/settings-admin-page

Conversation

@gnugomez

@gnugomez gnugomez commented May 6, 2026

Copy link
Copy Markdown
Member

@gnugomez
gnugomez requested a review from netomi May 6, 2026 11:24
@netomi
netomi changed the base branch from main to read-only-mode May 12, 2026 18:40
@netomi
netomi merged commit 03d7caf into eclipse-openvsx:read-only-mode May 12, 2026
8 checks passed
netomi pushed a commit that referenced this pull request May 15, 2026
netomi pushed a commit that referenced this pull request May 22, 2026
netomi pushed a commit that referenced this pull request May 26, 2026
netomi added a commit that referenced this pull request May 27, 2026
* add FeatureFlagService and aspect for read-only mode, add MutatingOperation annotation to indicate endpoint that modify the DB, add annotation to all relevant endpoints

* fix warning with existing aspect and add fixme for broken aspect

* refactor to SettingsService in package settings

* feat(admin): adding settings page (#1818)

* add api endpoints, rename feature flag to settings

* consistently use settings instead of feature flags, reduce notification timeout, add label

* bump jedis to 7.4.1 and refactor channel listener to a common base class

* add update listener to SettingsService

* fix ide warnings

* add persistency layer for settings

* add expiry for setting cache

* update setting key to trigger rebuild

* feat(admin/settings): adding friction on save

* improve error handling

* check for read-only mode in relevant background jobs

* add changelog entry

---------

Co-authored-by: Jordi Gómez Hidalgo <[email protected]>
Co-authored-by: Jordi Gómez <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants