Skip to content

feat(fe): integrate question shortening API #69

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Feb 6, 2025
Merged

feat(fe): integrate question shortening API #69

merged 3 commits into from
Feb 6, 2025

Conversation

@cjeongmin
Copy link
Contributor

@cjeongmin cjeongmin commented Feb 6, 2025

개요

질문 축약 API를 연결했습니다.

이슈

@cjeongmin cjeongmin requested review from shl0501 and yu-yj215 February 6, 2025 08:10
@cjeongmin cjeongmin self-assigned this Feb 6, 2025
Copilot AI review requested due to automatic review settings February 6, 2025 08:10
Copilot AI reviewed Feb 6, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • apps/client/src/features/create-update-reply/ui/CreateReplyModal.tsx: Evaluated as low risk

shl0501
shl0501 approved these changes Feb 6, 2025
View reviewed changes
Copy link
Collaborator

@shl0501 shl0501 left a comment

Choose a reason for hiding this comment

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

좋은 API에 대한 고민이 많았었는데, 정민님 덕분에 잘 연결한 예시에 대해 배우고 갑니다.
LGTM

yu-yj215
yu-yj215 approved these changes Feb 6, 2025
View reviewed changes
Copy link
Contributor

@yu-yj215 yu-yj215 left a comment

Choose a reason for hiding this comment

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

LGTM

@cjeongmin cjeongmin merged commit a72b9e3 into main Feb 6, 2025
2 checks passed
@cjeongmin cjeongmin deleted the feat(fe)/shorten branch February 6, 2025 08:14
@cjeongmin cjeongmin mentioned this pull request Feb 6, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@yu-yj215 yu-yj215 yu-yj215 approved these changes

@shl0501 shl0501 shl0501 approved these changes

Assignees

@cjeongmin cjeongmin

Labels

Feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feat]: 질문 축약 API 연동

4 participants

@cjeongmin @yu-yj215 @shl0501