-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Change HostEnvironment to use IHostingEnvironment #3
Labels
Comments
Infact, we should use IApplicationEnvironment on the HostEnvrionment, but keep that abstract and have IHostingEnvrionement for webapps. note: need to make sure that app_data does not move paths, we don't want it under wwwroot. |
DrewScoggins
added a commit
to DrewScoggins/OrchardCore
that referenced
this issue
Dec 9, 2022
Skrypt
pushed a commit
that referenced
this issue
Jan 25, 2024
^ This is the 1st commit message: Add GetCulture() extension method ^ This is the commit message #2: Cleanup ISmsService (#15142) ^ This is the commit message #3: Fix TheAdminTheme layout margin and padding (#15143) ^ This is the commit message #4: Fix SectionDisplayDriver prefix (#15123) ^ This is the commit message #5: Prefill template name when creating a template. (#15145) ^ This is the commit message #6: Set the User Localization feature priority ^ This is the commit message #7: Fix issue with default culture not selected When currentUserCulture is null or supportedCulture doesn't contain currentUserCulture. ^ This is the commit message #8: Update the height of the admin content (#15153) ^ This is the commit message #9: Eliminate the anti-discovery pattern in Elasticsearch (#15134) ^ This is the commit message #10: Renaming and cleaning up search services (#15156) ^ This is the commit message #11: mkdocs-material 9.5.5
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
IHostingEnvironment is for hosting webapplications. We should be using that rather that the fall back of IApplicationEnvrionment
The text was updated successfully, but these errors were encountered: