endpoint 이슈

[de-novo]

이슈

현재 api endpoint가 restful 하지않아 확장성에 문제가 있을 것으로 예상됩니다.

:GET /er/hospitals/current/illness을 예시로 들겠습니다

• url의 중복된 정보

  • 현재 시스템에서는 er과 hospitals는 중복된 정보를 담고있다고 생각합니다. -> hospitals를 제거해야 restful한것 같습니다!

• 메인 리소스의 위치

  • api가 다루고자하는 메인 리소스는 앞에 있어야 한다고 생각합니다.
  • 현재 api는 illness 즉 질병을 다루고있기에 맨뒤가아닌 맨앞에 가야할 것 같습니다.

• url에 불필요한 정보 (상태 or 행동)

  • restful api에서는 api에 상태 또는 행동을 명시하지않는 것이 좋다고 알고 있습니다.
  • 현재 current는 illness의 상태를 나타내는 것으로 판단되어 빼도 될 것 같다고 생각됩니다.

개선

/er/hospitals/current/illness -> /er/illnesses

:GET /er/illnesses 현재 응급실(로그인되어있는 응급실)에대한 질병들을 모두 제공해줘 (DELETE 제외)

:GET /er/illnesses?status=ACTIVE 현재 응급실(로그인되어있는 응급실)에대한 치료가능한(ACTIVE) 질병들을 제공해줘

:GET /er/:er_id/illnesses (외부전용 api) er_id인 응급실의 질병정보를 제공해줘

:GET /er/:er_id/illnesses?status=ACTIVE er_id인 응급실의 치료가능 질병정보를 제공해줘

:PATCH /er/illnesses 현재 응급실(로그인되어있는 응급실)에 대한 질병정보를 수정할래

질병뿐 아닌 장비관련해서도 endpoint수정이 필요할것으로 보입니다.

[de-novo]

19 에대한 이슈입니다

[YJ-AnthonyJo]

current는, 상태가 아니라, 현재 병원을 나타내는 용어로 사용되었습니다. 제가 이전에 이부분 까먹고 말씀 안드렸네요.. 제가 생각할때 외부에서 조회할때 er_id가 들어가듯 current가 들어가야 통일성이 있다고 생각해서 작업했습니다. 혹시 이 상태에서 current가 부적절한것 같으면 피드백주세요! pr: #26

[YJ-AnthonyJo]

equipment pr: #27

[YJ-AnthonyJo]

department는 작업하신 부분관련해서 상의가 필요할까요?