Skip to content
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

should Authentication process be split into two processes? #8

Open
mattjbrent opened this issue Feb 12, 2020 · 1 comment
Open

should Authentication process be split into two processes? #8

mattjbrent opened this issue Feb 12, 2020 · 1 comment
Assignees
Labels
status: information needed Further information is requested

Comments

@mattjbrent
Copy link
Collaborator

Just reviewing the work I committed for the authentication feature. The process is split up into two indirect and direct. with correlating log in direct and indirect features.

Now im looking at it im wondering if this should be split up into two separate processes rather than just one, despite their obvious similarities, they are showing two separate paths, even though the end result is the same.

@mattjbrent mattjbrent added the status: information needed Further information is requested label Feb 12, 2020
@cellio
Copy link
Member

cellio commented Feb 12, 2020

The "direct" path is the user clicking on a "log in or sign up" link somewhere on the page (conventionally in the header, but you'll figure that out). The user is taking an explicit action.

The "indirect" path is the user trying to do something that only logged-in users can do, and thus we interject a login/signup workflow.

I made these different use cases because I didn't know if the design considerations would be different. For example, with the indirect path we need a place to put a "hey, you'll need to be logged in to do that" message. The direct path might offer some "what to do next" guidance that we wouldn't want for the indirect path ("stop interrupting me and just let me answer!" etc).

I suggest keeping them separate even if they'll share 90%. I assume we can do this with reuse and not with cloning stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: information needed Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants