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

feat: 원활한 채팅 기능 개발을 위한 초안 작업 #35

Merged
merged 23 commits into from
Nov 12, 2023

Conversation

galaxy4276
Copy link
Contributor

@galaxy4276 galaxy4276 commented Nov 7, 2023

Type

  • Feature
  • Fix

Implements issue SU-140

What & Why

간단한 퍼블리싱 작업을 진행하였고 핵심내용은 다음과 같습니다.

어플리케이션 내 MSW 시스템을 탑재하였습니다.

기존 작업방식이 API가 완성되지않았다면 최대한 존재한다는 가정상황에서 작업을 하는것으로 인지하였으며
해당 작업방식에서 좀 더 API 연동에있어 다음과같은 Mock 작업을 수행하지 않아도 되게끔 환경을 작성하였습니다.

  • 해당 PR이 반영 전에는 Mock 객체를 생성하고 받아오는 모든 일련의 과정을 수동으로 작업한다. ( 실제 API 개발 시 코드 교체까지 )

이제 위 부분의 공수를 최소화하고 빠르게 작업할 수 있습니다. (HTTP의 요청과 응답을 실제로 수행함으로써)

또한 애플리케이션이 복잡한 만큼 실제데이터로 충분한 테스트가 어려운 점을 고려하여 빠른 개발을 수행할 수 있게 해줍니다.

간단한 패칭도구를 작성하였습니다.

API 항목을 불러오기 위한 동일한 구조의 함수작성을 추상화하였습니다.

다음과 같은 화면 기능 작업

Nov-12-2023 05-23-00

Checklist

  • 지라 티켓이 있다면 이슈 번호를 매핑하셨나요?

Copy link

vercel bot commented Nov 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
stock2u-front ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2023 4:08pm

@galaxy4276 galaxy4276 changed the title feature: 원활한 채팅 기능 개발을 위한 초안 작업 feat: 원활한 채팅 기능 개발을 위한 초안 작업 Nov 7, 2023
src/components/domains/chat/ChatRoomPreview.tsx Outdated Show resolved Hide resolved
src/components/domains/chat/ChatRoomPreview.tsx Outdated Show resolved Hide resolved
src/components/domains/chat/ChatRoomPreview.tsx Outdated Show resolved Hide resolved
@galaxy4276 galaxy4276 marked this pull request as ready for review November 12, 2023 14:33
bear-bear-bear
bear-bear-bear previously approved these changes Nov 12, 2023
src/components/domains/chat/ChatRoomPreview.tsx Outdated Show resolved Hide resolved
src/main.tsx Outdated Show resolved Hide resolved
@galaxy4276 galaxy4276 merged commit de99a2f into develop Nov 12, 2023
5 checks passed
@galaxy4276 galaxy4276 deleted the feature/SU-140 branch November 12, 2023 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants