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

Support for NC 26 #557

Open
WeblinxIT opened this issue Apr 3, 2023 · 10 comments
Open

Support for NC 26 #557

WeblinxIT opened this issue Apr 3, 2023 · 10 comments

Comments

@WeblinxIT
Copy link

Nextcloud 26 is out and the twofactor gateway is not compatible yet.

Does it work with Nextcloud 26?

Thanks and regards

@Cieniek
Copy link

Cieniek commented Apr 24, 2023

Seems to work just fine on NC 26.

@vitormattos
Copy link
Contributor

Yes, I think that work fine and only will be necessary to send a new PR bumping the Nextcloud version at info.xml file and verify if will be necessary to do changes at GitHub Actions to run using server with version 26

@nursoda
Copy link
Contributor

nursoda commented May 28, 2023

Since NC27 is about to be released, it should be bumped to 27 right away. On my Arch instance and using NGINX, MariaDB, PHP 8.2 and SMS77, Authentication still works as expected.

@vitormattos
Copy link
Contributor

Already was bumped to 27 here:
b3f29b5

But this app haven't anymore a maintainer available to merge the PR's from community and release a new version.

Or you run the app from master branch doing all the necessary to build the app or wait by the next version.

@nursoda
Copy link
Contributor

nursoda commented May 30, 2023

@nextcloud: I may maintain and release this app. However, I'd only manage it (and try to fix if something breaks), but I will not develop it further. Please contact me if you're interested. You'd also have to decide whether to move it out of nextcloud namespace. Ass user, I expect all apps within that namespace to be "officialy supported my nextcloud"!

@vitormattos
Copy link
Contributor

I did a lot of contributions at this app but I was removed from Nextcloud organization and, because this, I lost the permission to merge PR's at this repository and follow contributing with this app because haven't a maintainer.

I think that this issue need to be reopened since my permissions to be a maintainer was removed:
#454

@nursoda
Copy link
Contributor

nursoda commented May 30, 2023

If you'd continue to maintain it, please do, that's far better than me mingling with it. Please reach out to @nextcloud Staff via E-Mail or phone solve this deadlock situation. I assume no staff is reading here…

@florianschroen
Copy link

@ChristophWurst is there any progress in making @vitormattos the maintainer (ref #454), so keeping this app up to date can be done? thx

@ChristophWurst
Copy link
Member

#454 reopened.

@develroo
Copy link

Yeah, FWIW, now you get a warning this app is no longer compatible with 26.x. Needs testing and a bump, else it will be broken in future versions.

Screenshot from 2023-08-24 11-44-01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants