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

Implemented new picker #4

Merged
merged 2 commits into from
Jul 24, 2017
Merged

Implemented new picker #4

merged 2 commits into from
Jul 24, 2017

Conversation

aschempp
Copy link
Member

@leofeyer leofeyer changed the base branch from master to hotfix/4.4.2 July 24, 2017 09:24
@leofeyer leofeyer merged commit 8c3262d into contao:hotfix/4.4.2 Jul 24, 2017
leofeyer pushed a commit that referenced this pull request Jun 14, 2021
Description
-----------

Alternative approach for response context as discussed in Core team call.
I guess there's no point in explaining it all here - I'll just document the final iteration once everything is done in the Contao dev docs ^^

Commits
-------

205dafc7 Reworked response context concept
786b50ca Merge branch '4.x' into feature/response-context-rework
4d1cf189 CS
d2151bb4 Fixed tests
aae10917 Implemented removing existing services and added support for interfaces
0a83cbe4 Added psalm docs
5f27ffce Make HtmlHeadBag final
5e75e386 CS
950e9bbe Also consider parent classes as aliases
fbe8206c Going back to a regular array and removed the remove() method
41b1c60f Added support for lazy services
8a35bc96 Fixed tests
0d7f8d93 CS

Co-authored-by: Leo Feyer <github@contao.org>
5b9706c4 Added failing test to show class issue
d37a2237 Merge pull request #4 from aschempp/feature/response-context-rework

Added failing test to show class issue
f0a1f353 Removed irrelevant assertion
1d29c95c Fixed tests
6399cc78 Cleanup
12556327 Fixed phpstan
6a1eebc4 CS
e90daf48 Added failing test to show alias being overridden
a5c95893 Merge pull request #5 from aschempp/feature/response-context-rework

Added failing test to show alias being overridden
1cb74c7a Fixed tests
84af804d Removed leftover
62cb0af1 CS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants