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.

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

added recipe to use appsettings on server side. #351

Closed
wants to merge 5 commits into from

Conversation

RicoSaupe
Copy link
Contributor

#154

@isaacabraham @martinbryant
Please review and let me know if something is missing.

@isaacabraham
Copy link
Member

@RicoSaupe I'm not convinced that any of the packages and middleware is required. IConfiguration is available directly in ASP .NET and should be immediately accessible. You can get to IConfiguration and then use indexer syntax directly.

@RicoSaupe
Copy link
Contributor Author

you are correct. I started out differently and later decided to do less. I removed dependencies and moved my page to the patterns section you created. Please check again

@RicoSaupe RicoSaupe closed this Jan 18, 2024
@RicoSaupe RicoSaupe deleted the appsettingsdocs branch January 18, 2024 15:24
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