-
Notifications
You must be signed in to change notification settings - Fork 1
add UI for the webhook token #63
add UI for the webhook token #63
Conversation
} | ||
return s4() + s4() + '-' + s4() + '-' + s4() + '-' + | ||
s4() + '-' + s4() + s4() + s4(); | ||
} |
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 isn't technically a valid UUID (see https://en.wikipedia.org/wiki/Universally_unique_identifier#Version_4_.28random.29 for the bits that specify variant and version), but that doesn't really matter much for our purposes unless Django actually validates that.
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.
@jerith yup, django doesn't care. It only expects a 32 char string really
👍 assuming Django's okay with potentially "invalid" UUIDs. |
…webhook-token add UI for the webhook token
@@ -284,7 +284,6 @@ def abspath(*args): | |||
} | |||
DEBUG_TOOLBAR_PATCH_SETTINGS = False | |||
|
|||
SOCIAL_AUTH_SESSION_EXPIRATION = True |
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.
❤️
No description provided.