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

feat(auth): 인증 모듈 구현 #8

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

feat(auth): 인증 모듈 구현 #8

wants to merge 2 commits into from

Conversation

WhiteKiwi
Copy link
Owner

수정사항

  • 글로벌 prefix 설정 /api
  • 글로벌 versioning 설정 /api/v1
  • 글로벌 response converter 적용 => { data: { ... } }
  • 인증모듈 구현
    • login: /api/v1/auth/login
    • auth.module.ts에서 AuthenticatorModule을 교체하는 것으로 인증 방식 수정 가능, AuthDataStorage 구현체를 추가/수정하는 것으로 DataSource 추가 가능

고민

  • 데이터 수정 기능이 필요한가? DataStorage => DataLoader로 바꾸고 save 스펙 날릴까 고민중

@WhiteKiwi WhiteKiwi self-assigned this Mar 27, 2022
@WhiteKiwi WhiteKiwi requested a review from lkic1625 March 27, 2022 13:30
@WhiteKiwi WhiteKiwi force-pushed the feat/auth branch 2 times, most recently from 5e612f9 to edeb437 Compare March 27, 2022 14:21
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.

None yet

1 participant