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

refactor: ♻️ where42All api의 반환값 변경 #209

Merged
merged 6 commits into from
Jul 9, 2024

Conversation

niamu01
Copy link
Collaborator

@niamu01 niamu01 commented Jul 9, 2024

수정 및 작업 내용

  1. 어디있니 팀의 요청에 따라 반환 타입을 변경하였습니다.

Where42All (post) API의 응답:

[
  {
      "login": "intra-id1",
      "inoutState": "OUT",
      "cluster": "GAEPO",
  },
  {
      "login": "intra-id2",
      "inoutState": "IN",
      "cluster": "GAEPO",
  },
  {
      "login": "non-intra-id",
      "inoutState": null,
      "cluster": null,
  },
]

*클러스터가 추가될 것을 대비하여 클러스터를 포함하여 반환 (서초 삭제)
*admin 또는 잘못된 로그인으로 조회하는 경우 inoutState과 cluster를 모두 null로 반환

  1. 어디있니 팀의 요청에 따라 API의 베이스 url을 변경하였습니다.
    기존: /ext/where42/where42All
    변경: /ext/where42/where42/where42All

  2. 42가 운영되기 시작한 시간을 고정적으로 반환해주는 상수를 추가하였고(START_DATE),

  3. 날짜 및 시간 연산 파일 내 함수들의 주석이 제멋대로여서 상수 추가, 변수명과 주석을 수정하였습니다. (업보하나해결)

여담

저번에 확인한 어디있니 팀의 레포지토리가 다른 저장소였더라구요 🫢 소통의 중요성 다시 되새기고 갑니다...

@niamu01 niamu01 requested a review from enaenen July 9, 2024 15:00
@niamu01 niamu01 self-assigned this Jul 9, 2024
@niamu01 niamu01 added the refactor Need refactoring label Jul 9, 2024
@niamu01 niamu01 mentioned this pull request Jul 9, 2024
7 tasks
@niamu01 niamu01 changed the title refactor: ♻️ where42All api의 반환값 타입 새로운 요청대로 변환 refactor: ♻️ where42All api의 반환값 변경 Jul 9, 2024
@niamu01 niamu01 merged commit edad3c0 into develop Jul 9, 2024
3 checks passed
@niamu01 niamu01 deleted the refactor/where42api 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
refactor Need refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant