-
Notifications
You must be signed in to change notification settings - Fork 361
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
[CILogon] Don't make action a required field of CILogonOAuthenticator.allowed_idps follow-up #517
Conversation
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be 15.0.1 i figure, but right now the migration docs mentions only 15.0.0.
I suggest replacing 15.0.0 with 15.0.1 without further notes pretty much.
Co-authored-by: Erik Sundell <erik.i.sundell@gmail.com>
Thanks a lot @consideRatio! I believe I have addressed your comments. After the merge of this PR, I believe we should go for a 15.0.1 release. I also opened #518, but maybe that needs more discussion and should be released later? What do you think? |
@GeorgianaElena yepp unless its another bugfix, let's not include it before 15.0.1 is released. Want to go for a release? I merged a bigbucket bugfix pr just now as well btw. |
Yep! Let's see how it goes. I'll open up a PR with the updated changelog shortly |
I agree! |
Follow-up to #516. Without this, it's still impossible to don't specify an
action
.This also adds a test and documents this option.
Ref: #514