PHORAX / formhandler

Fork of TYPO3 extension «Formhandler»
GNU General Public License v2.0
36 stars 54 forks source link

Update FormHandler for Typo3 v11 #83

Closed tlueder closed 2 years ago

tlueder commented 2 years ago

Hi,

This is still a work in progress but I would like to get feedback as early as possible and also this patch is already a big one.

System setup:

What have I done:

Whats new:

What works:

Tested Validators:

Tested Finishers:

Todo:

Any help and input is much appreciated.

Also this can be broken up into separate pull requests if needed.

Thanks

pulponair commented 2 years ago

@tlueder Thanks for the huge amount of work you put into this that is awesome!. Would you mind splitting this PR into smaller chunks each addressing its own issue?. E.g. If you would outsource the "Remove swiftmailer" - part (https://github.com/PHORAX/formhandler/pull/83/commits/4aedaf9cfb29fcd02ddadbfe1e4e618d966dbbc2) in a seperate PR that would be probably merged right away. //cc @7elix

tlueder commented 2 years ago

Hi @pulponair , sure thing.

I'll prepare smaller PRs and group them by bug fix, clean up, new feature and so on. This big PR was meant as a look at this and tell me what you think about it.

Another question is about the typing. In this state formhandler needs php 8.0 would that be ok? Typo3 11 has a min php version of 7.4

pulponair commented 2 years ago

@tlueder Minimum php version should play along with the TYPO3 11 requirements. So php7.4 :)