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

Allow installation with TYPO3v10 LTS #1

Merged
merged 2 commits into from Sep 2, 2020
Merged

Allow installation with TYPO3v10 LTS #1

merged 2 commits into from Sep 2, 2020

Conversation

mbrodala
Copy link
Contributor

@mbrodala mbrodala commented Sep 2, 2020

No description provided.

The annotations of HTTP classes have been fixed TYPO3 v10 LTS with commit:
https://review.typo3.org/c/Packages/TYPO3.CMS/+/61558

Signed-off-by: Benjamin Franzke <benjaminfranzke@gmail.com>
@bnf
Copy link
Owner

bnf commented Sep 2, 2020

Will merge for sure, although the factories are natively available in v10.4, as I upstreamed them in https://review.typo3.org/c/Packages/TYPO3.CMS/+/61558

Still, out of interest: any special usecase that requires this package to be v10.4 compatible?
Maybe an extension that should be v9 and v10 LTS compatible?

@bnf bnf merged commit d9139bc into bnf:master Sep 2, 2020
@mbrodala mbrodala deleted the patch-1 branch September 2, 2020 12:31
@mbrodala
Copy link
Contributor Author

mbrodala commented Sep 2, 2020

Yes, I'm currently working on compatibility for TYPO3v10 in https://github.com/pagemachine/typo3-mail-css-inliner which uses your package for tests. It's good if I can keep dependencies unchanged between TYPO3 versions.

Also thanks for the quick release. :-)

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.

None yet

2 participants