Skip to content

Removed dependency on IServiceProvider from endpoint and action filter constructors.#26

Closed
mgernand wants to merge 0 commit intoSharpGrip:release/1.4from
mgernand:release/1.4
Closed

Removed dependency on IServiceProvider from endpoint and action filter constructors.#26
mgernand wants to merge 0 commit intoSharpGrip:release/1.4from
mgernand:release/1.4

Conversation

@mgernand
Copy link
Contributor

The service provider is available via the HTTP context.

@mvdgun
Copy link
Member

mvdgun commented Dec 14, 2023

Hi @mgernand, I just merged a couple of pull requests and unfortunately your pull request has conflicts now :(. From the looks of it it's shouldn't be very hard to resolve.

@mgernand
Copy link
Contributor Author

No problem. I'll take a look tomorrow. 🙂

@mvdgun
Copy link
Member

mvdgun commented Dec 14, 2023

Great! Could you also remove this. when accessing class members? The whole code bases doesn't use it and I want to keep things consistent.

@mgernand mgernand closed this Dec 15, 2023
@mgernand mgernand deleted the release/1.4 branch December 15, 2023 10:51
@mgernand mgernand restored the release/1.4 branch December 15, 2023 10:51
@mgernand mgernand deleted the release/1.4 branch December 15, 2023 10:51
@mgernand
Copy link
Contributor Author

@mvdgun The tests of the current release/1.4 branch has errors. So I will wait until those are fixed and will open another PR then.

@mvdgun
Copy link
Member

mvdgun commented Dec 16, 2023

@mgernand I just fixed the tests in the release/1.4 branch

@mgernand
Copy link
Contributor Author

@mvdgun I just opened PR #30 with the changes from this PR (and without this prefixed).

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