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

결과 조회를 개선한다 #33

Closed
2 tasks done
viiviii opened this issue Mar 8, 2023 · 0 comments · Fixed by #48, #53 or #58
Closed
2 tasks done

결과 조회를 개선한다 #33

viiviii opened this issue Mar 8, 2023 · 0 comments · Fixed by #48, #53 or #58

Comments

@viiviii
Copy link
Owner

viiviii commented Mar 8, 2023

문제

  • reusable actions에 대해 알아보니 schedule은 caller측에서 지정해야 했다.
  • 난 called측에서 제공할 수 있을거라 생각하여 날짜 범위를 직전 회차의 일주일로 개발했으나 이 부분을 수정하는게 쓰기 편할 것 같다.

고민한 부분

결과 조회, 기간을 어떻게 제공할지

  • Case A: 시작일, 종료일 날짜를 사용자가 직접 지정
  • Case B: 오늘, 일주일, 한달 3가지 옵션으로 제공
  • Case C: 직전 회차 조회

고민 이유

  • 날짜 입력 시 git actitons yml은 리눅스 + actions 명령어를 알아야하므로 기존 코드에 비해 사용자가 사용하기 쉽지 않다

결정

사용자가 편한 것으로 제공해라

  • 내가 계속 고민된 이유는 어중간해서였던 것 같다
    • 편하게 제공할거면 [직전 회차 조회]로 제공하는게 낫다

변경 및 개선할 부분

  • 직전 회차 조회
    • 이전 판매일 시작, 종료일인 일요일~토요일까지를 조회한다
  • 조회 결과는 예전처럼 Table로 리턴
    • 그러면 조회 결과가 없는건 메세지로 보여주든 예외를 던지든 클라이언트 코드에서 결정할 수 있다

중간에 바꾼 부분

  • 조회 기간 직접 입력 기능 제거
    • 이유는 동행복권 사이트의 페이징 사이즈가 10이라 한달이면 2번의 API를 호출해야된다
    • 1년이면 48번, 10년이면 480번인데 내 서버도 아닌데 너무 막 호출하는 것 같아서 제거했음
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment