OCA
[MIG] auth_oauth_autologin: Migration to 17.0
@Loregs2 /web/login?no_autologin not working at the moment similar to #627
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
This module works for me. What needs to be done to have it merged?
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.
This module works for me. What needs to be done to have it merged?
You can check ‘Approve’ in the Review changes section of the files, thanks.
@Loregs2 fix with ?no_autologin is not included unless it's not needed anymore? https://github.com/OCA/server-auth/pull/627/files#diff-f3238b0909b811978b1f783e9dde0a41031587b3de012f0356a68c81c855e043R15
Otherwise it looks ok
@Loregs2 fix with ?no_autologin is not included unless it's not needed anymore? https://github.com/OCA/server-auth/pull/627/files#diff-f3238b0909b811978b1f783e9dde0a41031587b3de012f0356a68c81c855e043R15
Otherwise it looks ok
You were right, I have added that change and the ?no_autologin is now working. Thank you.
What a great day to merge this nice PR. Let's do it! Prepared branch 17.0-ocabot-merge-pr-658-by-dreispt-bump-nobump, awaiting test results.
Congratulations, your PR was merged at 27778a47ba4bbf5d9f34ee8b62e3269f45a1e95f. Thanks a lot for contributing to OCA. ❤️