Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Config declaration legacy key#7350

Merged
amercader merged 5 commits into
masterfrom
config-declaration-legacy-key
Jan 24, 2023
Merged

Config declaration legacy key#7350
amercader merged 5 commits into
masterfrom
config-declaration-legacy-key

Conversation

@smotornyuk
Copy link
Copy Markdown
Member

@smotornyuk smotornyuk commented Jan 24, 2023

Fixes #7084 / Enchantment for #7280

Add a legacy_key attribute to the config declaration.

When the config option is missing, but the corresponding legacy_key option is available, this "legacy" value is used(accompanied by a deprecation warning)

Note
I'll create a backport PR as soon as this PR get approved

@smotornyuk smotornyuk force-pushed the config-declaration-legacy-key branch from 9abc99c to 5330845 Compare January 24, 2023 09:16
@amercader amercader self-assigned this Jan 24, 2023
@amercader amercader merged commit 5330845 into master Jan 24, 2023
@amercader amercader deleted the config-declaration-legacy-key branch January 24, 2023 12:21
@amercader
Copy link
Copy Markdown
Member

Thanks @smotornyuk this is great, I backported it to 2.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

route_after_login is not documented

2 participants