ResponseEntity 리턴값은 ResponseEntity.ok() 이런식으로 하지 말고
ResponseEntity.status(HttpStatus.상태코드이넘).build()로 통일
add edit remove 대신 create update delete 로 통일
왜냐면 메서드명은 개발자가 보는거고
사용자는 유알엘을 보니까
에러코드
중복될 경우 - 409 conflict
없을 경우 - 404 not found
서버가 아프면 - 500
권한 없으면 - 401
🌱 TODO (기한은 내일 오전까지)
🚨 공통
prod.yml에 인플럭스디비 url, token 넣어야함
서비스 구현체에서 dto로 맵핑하는 부분 스트림으로 이쁘게 작성하기
리포지토리에서 리스트 가져올 때 n+1문제 감안하고 쿼리 여러 개 날릴지 아니면 쿼리 하나 날릴지도 고민해보기
리턴값은 ResponseEntity.status(HttpStatus.상태코드이넘).build()로 통일
🇰🇷 승진
PUT /api/sensors/modbus/health 자바독 작성
🍞 준영
MonthlyElectricityController @Operation 작성
MonthlyElectricityController , PlaceController에서 엔티티→dto로 바꾸고잇는데 서비스단에서 하도록 수정
placecontroller에 자바독 추가
🐱 세연
채널컨트롤러에서도 aop(@VerifyOrganization) 쓸 수 있도록 수정
MqttSensorController에서도 aop쓰도록 하고 서비스단에서 fetchSensorWithOrgValidation 메서드는 삭제하기
🌱 바꾼 url
GET /api/admin/budget-histories
GET /api/admin/check-code
PUT /api/admin/organization-name
PUT /api/admin/organization-code
PUT /api/admin/role
GET /api/sensors/modbus/channels/by_place -> /api/sensors/modbus/channels/by-place
PUT /api/sensors/modbus/{sensorId}/update -> PUT /api/sensors/modbus/{sensorId}
PUT /api/sensors/modbus/work -> PUT /api/sensors/modbus/health
MonthlyElectricityController 리턴타입이 ResponseEntity로 바뀌었음
PUT /api/sensors/mqtt/{sensorId}/update -> PUT /api/sensors/mqtt/{sensorId}
POST /place 리턴타입이 ResponseEntity로 바뀌었음
PUT /api/place/update -> PUT /api/place
DELETE /api/place/delete -> DELETE /api/place
🌱 고민
모든 dto @Schema 적어야함?
모든 서비스에 자바독 써야함?
🧚 2일차
TODO
공통
모든 프론트 컨트롤러에 모델에 UserInfo 넣어주는 부분 지우기 (인터셉터로 따로 뺐음)
url, dto 바뀐것 모두 적용
feature/first-refactoring 브랜치에서 각자 브랜치 파서 리팩토링&수정 후 feature/first-refactoring 에 머지하기
🧚♀️ 1일차
🌱 규칙
모든 url은 올 소문자, 대시(-)로 통일
ResponseEntity 리턴값은 ResponseEntity.ok() 이런식으로 하지 말고
ResponseEntity.status(HttpStatus.상태코드이넘).build()로 통일
add edit remove 대신 create update delete 로 통일 왜냐면 메서드명은 개발자가 보는거고 사용자는 유알엘을 보니까
에러코드
중복될 경우 - 409 conflict
없을 경우 - 404 not found
서버가 아프면 - 500
권한 없으면 - 401
🌱 TODO (기한은 내일 오전까지)
🚨 공통
prod.yml에 인플럭스디비 url, token 넣어야함
서비스 구현체에서 dto로 맵핑하는 부분 스트림으로 이쁘게 작성하기
리포지토리에서 리스트 가져올 때 n+1문제 감안하고 쿼리 여러 개 날릴지 아니면 쿼리 하나 날릴지도 고민해보기
리턴값은 ResponseEntity.status(HttpStatus.상태코드이넘).build()로 통일
🇰🇷 승진
PUT /api/sensors/modbus/health 자바독 작성
🍞 준영
MonthlyElectricityController
@Operation 작성MonthlyElectricityController
,PlaceController
에서 엔티티→dto로 바꾸고잇는데 서비스단에서 하도록 수정placecontroller에 자바독 추가
🐱 세연
채널컨트롤러에서도 aop(
@VerifyOrganization
) 쓸 수 있도록 수정MqttSensorController에서도 aop쓰도록 하고 서비스단에서 fetchSensorWithOrgValidation 메서드는 삭제하기
🌱 바꾼 url
🌱 고민
모든 dto @Schema 적어야함?
모든 서비스에 자바독 써야함?
🧚 2일차
TODO
공통
모든 프론트 컨트롤러에 모델에 UserInfo 넣어주는 부분 지우기 (인터셉터로 따로 뺐음)
url, dto 바뀐것 모두 적용
feature/first-refactoring
브랜치에서 각자 브랜치 파서 리팩토링&수정 후feature/first-refactoring
에 머지하기일요일 또는 월요일에 문제 없이 잘 돌아가도록 수정해와서 develop에 머지하기로 함