-
Notifications
You must be signed in to change notification settings - Fork 3
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
Replace requirement of typo3-oauth2-client #20
Comments
Thanks for the feedback @hirnschmalz! Yes, I'm aware of the current situation with the waldhacker repo. I'm still waiting for a response of in2code - if they are added as maintainer, the vendor would not change. Could you open a PR with you changes please? |
Thanks, I just tested your PR #22. Unfortunately in2code did not adjust the frontend login for v12. I'll stick to my fork until a stable version of ext-oauth2-client is released. |
is there any news about the situation? |
@secured Unfortunately, the guys from co-stack are not able to create a new release that contains my fixes. I don't get the point of moving to GitLab, changing the vendor name, adding new author names, but not releasing a new version. We need to wait. Maybe I will publish a version since I cannot see a roadmap for a v13 release. |
@maikschneider I'm a bit confused, as far as I can see your code was merged into the devlopment branch: Or am I missing something? |
@hirnschmalz Yes, they merged it. But I cannot use it as dependency as long as they are not tagging it. |
@maikschneider a OK, I see. But you could require |
Requiring "dev-anything" as dependency in a third-party package would force the main project to lower the minimum-stability. Yeah, this is not a good idea. I managed to update the oauth-client to v13: We're currently refactoring the tests for real acceptance tests - I think we will release a version by the end of September. |
@maikschneider I tried to install your v13 branch of both extensions. Unfortunately the requirements are wrong (as far as I can tell). The requirement in "waldhacker/typo3-oauth2-client": "dev-feature/v12-compatibility-1 || v13.x-dev", The composer docs say:
Took me a bit to figure this out 😉 |
Regarding to waldhacker/ext-oauth2-client#19 (comment) the extension will not be maintained by to original creator anymore.
Could you change the requirement in composer.json from
"waldhacker/typo3-oauth2-client",
toin2code/typo3-oauth2-client-fork
?The text was updated successfully, but these errors were encountered: