Skip to content

Add Tenant in non-linear way #1027

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

Merged
merged 8 commits into from
Sep 13, 2021
Merged

Conversation

dvaldivia
Copy link
Collaborator

This PR mades the the create tenant screen non linear, now the first screen will asks for the basic and then the user can go straight to any other step.

Screen Shot 2021-09-10 at 12 45 08 PM

Screen Shot 2021-09-10 at 12 49 13 PM

Signed-off-by: Daniel Valdivia 18384552+dvaldivia@users.noreply.github.com

@dvaldivia dvaldivia self-assigned this Sep 10, 2021
@adfost
Copy link
Contributor

adfost commented Sep 10, 2021

When you log in, it immediately goes to the create tenant page

adfost
adfost previously approved these changes Sep 10, 2021
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
bexsoft
bexsoft previously approved these changes Sep 13, 2021
Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Some changes might be required to make it more clear. this will be done in the future in small pieces

Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Some changes might be required to make it more clear. this will be done in the future in small pieces

@dvaldivia dvaldivia merged commit 0fdf5ee into minio:master Sep 13, 2021
@dvaldivia dvaldivia deleted the add-tenant-nonlinear branch September 13, 2021 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants