Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

AppConfig is not recommended everywhere #12221

Open
SMillerDev opened this issue Sep 21, 2024 · 4 comments
Open

AppConfig is not recommended everywhere #12221

SMillerDev opened this issue Sep 21, 2024 · 4 comments
Assignees
@ArtificialOwl
Labels
1. to develop manual: developer

Comments

@SMillerDev
Copy link

Since nextcloud 29 there is: AppConfig
But if you're looking at the configuration docs there is no mention of this, and implementing those recommendations will set you up to use deprecated methods that are going away.

The documentation should probably recommend AppConfig over IConfig.
@ChristophWurst
Copy link
Member

Indeed the new APIs were documented with #11617 but existing examples need adjustment.

@ChristophWurst
Copy link
Member

This needs backports for 30 and 29

@Duo123418
Copy link
Contributor

Hi, I would like to work on this issue by updating the documentation to reflect the recommended usage of AppConfig in place of IConfig. Could I proceed with this?

@SMillerDev
Copy link
Author

As far as I know anyone can make a pull request here, so I'd say you can proceed.

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

@ArtificialOwl ArtificialOwl

Labels
1. to develop manual: developer
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@ArtificialOwl @ChristophWurst @SMillerDev @joshtrichards @provokateurin @Duo123418