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

WS chat에 Message ID를 추가합니다. #86

Merged
merged 3 commits into from
Aug 10, 2024
Merged

Conversation

litsynp
Copy link
Contributor

@litsynp litsynp commented Aug 10, 2024

클라이언트에서 보낸 메시지와 대조, 필터링하기 위한 Message ID를 추가합니다.
클라이언트에서 보낸 그대로 보냅니다.

@litsynp litsynp added the feat New feature label Aug 10, 2024
@litsynp litsynp requested a review from JoeCP17 August 10, 2024 06:56
@litsynp litsynp self-assigned this Aug 10, 2024
@@ -19,7 +19,7 @@ jobs:
- uses: actions/checkout@v4

- name: Set up database
run: docker-compose -f docker-compose-test.yml up -d
run: docker compose -f docker-compose-test.yml up -d
Copy link
Contributor Author

Choose a reason for hiding this comment

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

GitHub에서 v1을 deprecate 시켜서 변경합니다.
https://docs.docker.com/compose/migrate/

Copy link

@kevinkim2586 kevinkim2586 left a comment

Choose a reason for hiding this comment

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

LGTM!

@litsynp litsynp merged commit 7e1f294 into epic/chat Aug 10, 2024
1 check passed
@litsynp litsynp deleted the feat/message-id branch August 10, 2024 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants