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.

Sign up for GitHub

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

Jump to bottom

Move oauth.ts from Services to User Management #150

Closed
kalebhouck opened this issue Feb 18, 2021 · 0 comments · Fixed by #170
Closed

Move oauth.ts from Services to User Management #150

kalebhouck opened this issue Feb 18, 2021 · 0 comments · Fixed by #170
Labels
good first issue Good for newcomers

Comments

@kalebhouck
Copy link
Collaborator

kalebhouck commented Feb 18, 2021 •
edited
Loading

Reason

For clarity and better code organization.

Design

Consolidate authentication code in one spot. Move oauth.ts from Service package to User Management package.

Impact

Minimal impact
@DnOberon DnOberon added the good first issue Good for newcomers label Feb 18, 2021
@jerenmb jerenmb linked a pull request Mar 11, 2021 that will close this issue
Domain, Repository, Mapper Refactor #170
Merged
33 tasks
DnOberon pushed a commit that referenced this issue Dec 9, 2022
@browjm4
clone ontology fix (#150)
b28a521
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Domain, Repository, Mapper Refactor idaholab/Deep-Lynx
2 participants
@DnOberon @kalebhouck