-
Notifications
You must be signed in to change notification settings - Fork 48
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
Fully-Automatic Tenant Registration Flow #584
Comments
I think this would be (in the Innkeeper plugin):
If this were to be supported in the Tenant UI would just need some tweaks to detect if the password is returned in the reservation step and appropriate screen text. |
I think we can just go with the regular check-in flow, just short-circuit the approval. The password is currently being sent out via email, isn't it? I'm looking at triggering the exact same code as the one that is executed when the |
On second thought, I think @loneil you're right: we will probably want to display the password on-screen for immediate use. |
Yeah exactly, that's the flow I mean in bullet point 2 above. The password is just returned by the approve endpoint currently, so it would be returned by the 1st POST instead if this auto-flag is on. |
Sorry was dragging my tickets around and picked up this one, not sure if this and #587 should be in done |
For situations such as what is described here, it would be nice to have a fully-automatic tenant approval option that will not require the innkeeper to manually approve one by one each registration request.
Acceptance Criteria:
Add an environment variable to be set at deployment time that, if present and set to
true
, automatically approves the tenant registration request.The tenant will still be responsible for completing the
check-in
step and obtaining the credentials to access/control their agent.The text was updated successfully, but these errors were encountered: