-
-
Notifications
You must be signed in to change notification settings - Fork 100
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
docs: add documentation for post_authenticate signal #351
Conversation
Thanks! This looks good to me |
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 is great to have, thank you! Let me know what you think of my suggestions. If you don't have time to implement them, I can help.
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.
i can implement those, no problem. should be able to get to it this weekend if not before. thanks for the suggestions!
@tim-schilling i think i got all those - let me know if that looks ok! |
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.
Excellent, thank you a ton!
I fumbled around for a while trying to figure out how to execute some custom code after a user is created/authenticated via SSO before I found the
post_authenticate
signal in the source code. I didn't see it in the docs, and having it in there would have saved me some time.I didn't really think it fit into any of the existing sections, but if you'd prefer it to be in there (maybe under installation, I guess?), let me know and I can move it.