Lecture test/#117

[gahyuun]

📌 작업 내용

구현 내용 및 작업 했던 내역

• drawer 닫히는 현상 해결
• 폐강과목 처리하기
• lecture search 관련 타입 변경

🔊 도움이 필요한 부분

• 항상 발생했던 건 아닌데, drawer 내부를 클릭하면 5번 중 한 번은 자기 맘대로 닫히는 현상이 있었습니다 onRelease가 실행되고 toggle이 실행되는 걸로 확인을 했습니다. onRelease가 제가 보았을 땐 특별한 역할을 하지 않아서 지웠는데 혹시 onRelease가 꼭 필요한 역할을 한다면 말씀해주세요!
• 백엔드랑 소통하다가 알게 된 사실인데 json변환라이브러리로 인해 isTaken 속성을 taken으로 변경했다고 전달받았습니다. lecture search에 있는 isTaken은 수정했습니다 result page에서 isTaken 속성이 존재하던데 한 번 확인해보세요~!

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-fahjxbhmst.chromatic.com/

[seonghunYang]

수고하셨습니다. branch의 이름이 'lecture-test'인 이유는 해당 변경사항들이 테스트 서버와 연동하여 변경된 내용이기 때문일까요?

[gahyuun]

수고하셨습니다. branch의 이름이 'lecture-test'인 이유는 해당 변경사항들이 테스트 서버와 연동하여 변경된 내용이기 때문일까요?

넵 원래 해당 브랜치에 테스트 서버랑 연결 작업까지 하려했는데, 진행하지 못했구 이후 다른 브랜치에서 진행할 것 같습니다!

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-ypqzbpjfnx.chromatic.com/

[yougyung]

onRelease는 현재 Drawer가 내려갈때 callback으로 호출시키고 있는 onClose(result page에서 다이아로그를 키고 끄는 경우 url의 변경)를 실행시켜주는 역할을 하고 있습니다. 따라서 이를 제거하는 것은 적절하지 않을 것으로 예상되고 발행되는 문제에 대해서는 저도 함께 확인해보겠습니다!

[gahyuun]

onRelease는 현재 Drawer가 내려갈때 callback으로 호출시키고 있는 onClose(result page에서 다이아로그를 키고 끄는 경우 url의 변경)를 실행시켜주는 역할을 하고 있습니다. 따라서 이를 제거하는 것은 적절하지 않을 것으로 예상되고 발행되는 문제에 대해서는 저도 함께 확인해보겠습니다!

제가 onRelease를 지우면서 말씀해주시는 side effect 발생에 대해 확인했습니다
그래서 제가 생각한 해결 방안은 노션에 올려두었습니다
https://www.notion.so/onRelease-f119f1cfac034315a7c76d787518e160?pvs=4
하지만 이게 최선의 방법인지, side effect가 없는지는 조금 확인이 필요할 것 같습니다 ( 생각보다 이슈 해결하기 어려워서요,,ㅠㅠ)
result랑 drawer을 언니가 작업해서 제일 잘 알 것 같은데 위 방안대로 문제 해결에 도움을 주실 수 있나요?

그리구 onRelease문제를 해결하는데 코드 수정이 좀 필요할 것 같아서 이 pr은 머지를 시키고 그 오류 해결로 새로운 pr을 올리는 건 어떨까요??

[yougyung]

그리구 onRelease문제를 해결하는데 코드 수정이 좀 필요할 것 같아서 이 pr은 머지를 시키고 그 오류 해결로 새로운 pr을 올리는 건 어떨까요??

넵 제시해준 방안 적용해서 PR올려둘게요 :)