search

pondersource / nextcloud-mfa-awareness

Make Nextcloud aware of whether the current user is logged in with Multi-Factor Authentication
MIT License
0 stars 2 forks source link

WIP: Mfaverified only zone #29

Closed mrvahedi68 closed 1 year ago

michielbdejong commented 1 year ago

What is this PR/branch for? The branch title is almost identical to the project title so it can describe almost anything. :)

mrvahedi68 commented 1 year ago

We have no changes in this project and only may add new submodules to that.

mrvahedi68 commented 1 year ago

It's not related to Milestone 5 and I use it for testing only. We can merge it and then I will use main branch for test.

michielbdejong commented 1 year ago

Great! I clicked "approve", you can go ahead and merge it now.

michielbdejong commented 1 year ago

Thanks for explaining, I now understand what this PR is and why it didn't have a clear name.

Agreed, if this branch is what you are using and nobody is using the main branch anymore, then this should become the main branch.

mrvahedi68 commented 1 year ago

Yes, agree with you. I think if we need to add some submodules for running inside the nextcloud and testing we can create another branch. The name of this branch is not suitable for this approach.