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: 어디있니 추가 API #207

Merged
merged 9 commits into from
Jul 9, 2024
Merged

✨ feat: 어디있니 추가 API #207

merged 9 commits into from
Jul 9, 2024

Conversation

niamu01
Copy link
Collaborator

@niamu01 niamu01 commented Jul 4, 2024

요청

어디있니 팀:
혹시 저희가 카뎃 이름 리스트를 파라미터로 보내면

{  
  이름 : intraId
  상태 : in / out
}

의 DTO로 보내주실 수 있는지 생각이 들었습니다.

  • 파라미터 -> body로 이야기 완료

추가한 API

POST 요청 속 body:
[ "intra-id1", "intra-id2", "intra-id3", ... ]

응답:

[
  {
      "login": "intra-id1",
      "inoutState": "OUT"
  },
  {
      "login": "intra-id2",
      "inoutState": "IN"
  },
  {
      "login": "intra-id3",
      "inoutState": null
  },
]

*반환 데이터는 순서가 중요하지 않다고 하여 비동기로 처리하였습니다.
*admin유저(bocal) 혹은 없는 로그인에 대한 inoutState는 null로 반환을 요청받았습니다.
*현재는 login, inoutState뿐만 아니라 기존의 타입인 cluster, tag_at도 함께 반환합니다.

변경

  1. 위에 언급한 예외처리를 위해 inoutState의 타입이 InOut에서 InOut | null 이 되었습니다.
  2. IdLoginDto 클래스에 is_admin 이 추가되었습니다.

어디있니 팀 작업 이후 변경 예정 작업

  1. Where42ResponseDto의 타입이 필요한 login과 inoutState만 남기기 (cluster, tag_at 삭제)
    a. 기존 api도 이 타입을 반환하여 함께 변경

여담

  • import 정렬은 PR에 안 뜨게 하려고 브랜치까지 나눴었는데 그만 머지해버렸습니다. 그냥 슥 지나가 주세요...!
  • inoutState랑 tag_at은 왜 같은 dto에 있는데 표기법이 왜 이렇죠...? 🫨 다음에 함수 이름 변경할 때 같이 갈아엎겠습니다.

@niamu01 niamu01 added the enhancement New feature or request label Jul 4, 2024
@niamu01 niamu01 requested a review from enaenen July 4, 2024 09:54
@niamu01 niamu01 self-assigned this Jul 4, 2024

const cards = await this.userService.findCardsByUserId(
user.user_id,
new Date('2019-01-01 00:00:00'),
Copy link
Collaborator

Choose a reason for hiding this comment

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

'2019-01-01 00:00:00' 와 같이 고정된 시작시간 같은경우에는 utils/common.constants.ts 에 정의해서 상수로 쓰는게 어떨까 싶습니다

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

기존 것 그대로 사용했는데 시작 날은 고정이니 그러는 게 좋겠군요 👍
이후에 리팩토링 브랜치에서 꼭 진행하도록 하겠습니닷

Copy link
Collaborator

Choose a reason for hiding this comment

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

기존에 보내주던 출/퇴 시간은 필요없나요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이름/상태만 필요하다구 하시더라구용

Copy link
Collaborator

@enaenen enaenen left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!! 👍
코멘트 확인해보시고 괜찮으면 수정 후 머지하시거나, 크게 상관없다고 생각되면 머지하시면 될거같습니다!

@niamu01 niamu01 merged commit 2aff8e0 into develop Jul 9, 2024
3 checks passed
@niamu01 niamu01 mentioned this pull request Jul 9, 2024
7 tasks
@niamu01 niamu01 deleted the addWhere42API branch July 9, 2024 20:04
@niamu01 niamu01 mentioned this pull request Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants