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: 채팅 팝업 오류 수정 #76

Merged
merged 3 commits into from
Aug 5, 2024
Merged

Feat: 채팅 팝업 오류 수정 #76

merged 3 commits into from
Aug 5, 2024

Conversation

eunji-0623
Copy link
Owner

🔎 작업 내용

  • 같은 기기에서 여러 아이디로 로그인 시 토큰이 변경 되지 않는 오류 수정
  • 채팅 입력 안하고 전송 버튼 클릭 시 제출되지 않도록 수정
  • 가끔 관리자->유저로 채팅 보낼 때 같은 메세지가 여러번 보내지던 현상 수정
  • 모바일 화면에서 top button 위치 수정

⚠️ 이슈

  • 배포 버전에서는 서버 응답 속도로 인해 채팅 불러오는 시간이 길어지는데.. 시간이 된다면 로딩 처리 추가해보겠습니다.
  • 관리자 채팅에서 이전 버튼 클릭 시 채팅방 마지막 메세지가 업데이트 되지 않는데 이것도 시간 될 때 수정해보겠습니다.

📢 전달사항

  • x

Copy link

netlify bot commented Aug 2, 2024

Deploy Preview for globalnomad-5-8 ready!

Name Link
🔨 Latest commit 8cee3a8
🔍 Latest deploy log https://app.netlify.com/sites/globalnomad-5-8/deploys/66ac718c61906e00081c895e
😎 Deploy Preview https://deploy-preview-76--globalnomad-5-8.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Aug 2, 2024

Deploy Preview for globalnomad-feature ready!

Name Link
🔨 Latest commit 8cee3a8
🔍 Latest deploy log https://app.netlify.com/sites/globalnomad-feature/deploys/66ac718c1e665000087db943
😎 Deploy Preview https://deploy-preview-76--globalnomad-feature.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@Byukchong Byukchong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다~

@eunji-0623 eunji-0623 merged commit 60d729b into master Aug 5, 2024
9 checks passed
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