Skip to content

Conversation

@ArtificialOwl
Copy link
Member

@ArtificialOwl ArtificialOwl commented Nov 3, 2025

This PR is a duplicate of #55477 which is locked in a strange state.


We use the appconfig/userconfig to store data others than real configuration value (ie timestamp and status) which should not be changed without a real knowledge of its implication.

This allow to define a config key in the Lexicon as INTERNAL:

	new Entry('test', ValueType::INT, 1, 'my test', lazy: false, flags: IAppConfig::FLAG_INTERNAL),

One of its application is to warn users that this is not a real config value.

image

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
@ArtificialOwl ArtificialOwl marked this pull request as ready for review November 3, 2025 12:47
@ArtificialOwl ArtificialOwl requested a review from a team as a code owner November 3, 2025 12:47
@ArtificialOwl ArtificialOwl requested review from icewind1991, leftybournes, salmart-dev and yemkareems and removed request for a team November 3, 2025 12:47
@ArtificialOwl ArtificialOwl merged commit 4a399a3 into master Nov 3, 2025
194 of 196 checks passed
@ArtificialOwl ArtificialOwl deleted the feat/noid/lexicon-internal-flag branch November 3, 2025 13:55
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.

4 participants