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

Comments for "The Ambient Composition Model" #4

Open
dotnetjunkie opened this issue Jul 11, 2019 · 2 comments
Open

Comments for "The Ambient Composition Model" #4

dotnetjunkie opened this issue Jul 11, 2019 · 2 comments

Comments

@dotnetjunkie
Copy link
Owner

@dotnetjunkie dotnetjunkie commented Jul 11, 2019

In the section below, you can leave your comments for this blog post: https://blogs.cuttingedge.it/steven/posts/2019/ambient-composition-model/.

Please be aware that a comment posted here will be shown in the comment section of the blog post, using JavaScript. This means that you should consider this a comment directly on my weblog—not as a post on GitHub.

This means that I might

  • Alter your comment—For instance, to correct spelling or make style and layout corrections
  • Delete your comment—In case I delete your comment, I will typically try to contact you to get your question answered. I can, for instance, delete your comment when I feel that the comment
    • Is not constructive
    • Is not related to the topic
    • Is too long
    • Is not helpful to most of my readers
@rexcfnghk
Copy link

@rexcfnghk rexcfnghk commented Dec 7, 2019

Hi Steven, thank you for your amazing insights. I have two questions about the
AmbientShoppingBasketContextProvider you have shown though.

  1. The AsyncLocal<T> class has two constructors but none of them takes a `Func constructor parameter. Is there a mistake in your example code?
  2. In the context of an ASP.NET Web API/MVC application, would the same ShoppingBasketDbContext be returned from this provider for a given request? Does it make a difference in an ASP.NET Core application?

Loading

@dotnetjunkie
Copy link
Owner Author

@dotnetjunkie dotnetjunkie commented Dec 8, 2019

Hi @rexcfnghk, thank you for noticing. This was indeed an error. I updated the blog post accordingly.

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants