search

sandstorm / NeosTwoFactorAuthentication

Extend the Neos Backend Login with 2FA
MIT License
12 stars 8 forks source link

Incompatible with PHP < 8 #2

Closed bwaidelich closed 2 years ago

bwaidelich commented 2 years ago

Hi,

the composer manifest claims that this package is compatible with PHP 7.4, but it is not (see https://github.com/sandstorm/NeosTwoFactorAuthentication/blob/e58561bf3dedf5ea08c4999a0e4a03de264a0728/Classes/Domain/Model/Dto/SecondFactorDto.php#L24 for example).

Great package by the way! :)

Pingu501 commented 2 years ago

Hey @bwaidelich

glad you like this package! You are totally right, this is php8 syntax. Fixed this and did a release. With version 1.0.1 this is fixed :)