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

Dependency resolution in middlewares doesn't work as of ecdfd16 #8

Closed
jasper-d opened this issue Jan 26, 2015 · 4 comments
Closed

Dependency resolution in middlewares doesn't work as of ecdfd16 #8

jasper-d opened this issue Jan 26, 2015 · 4 comments
Assignees
Labels

Comments

@jasper-d
Copy link

Running latest version in master (ecdfd16) from VS CTP 5 using k web results in a NRE inOrchardShellHostMiddleware. ShellSettings doesn't seem to be registered anywhere.
I tried to fix it, but it did more harm than good.

Similar problem in OrchardRouterMiddleware. IRouteBuilder is registered in ShellContainerFactory, but ShellContainerFactory.CreateContainer([...]) isn't called yet when the middleware is executed.

Environment: VS CTP 5, 1.0.0-beta3-10921 CLR x86

@Jetski5822 Jetski5822 self-assigned this Jan 26, 2015
@Jetski5822
Copy link
Member

Oddness - seems to be okay for me.

I am running on 1.0.0-beta3-10921 CLR x86

I load up OrchardVNext sln, compile, then open a command prompt with admin privileges. I then do "k run"

@Jetski5822
Copy link
Member

On the other hand, I dont think I checked in my dummy shell settings. - I have check in now, so can you do a pull?

@jasper-d
Copy link
Author

Pulled, k web, works. Thanks!

@Jetski5822
Copy link
Member

Welcome.

DrewScoggins pushed a commit to DrewScoggins/OrchardCore that referenced this issue Dec 1, 2023
The old version has known vulnerabilities.
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
Labels
Projects
None yet
Development

No branches or pull requests

2 participants