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: 이력서 수정 #37

Merged
merged 8 commits into from
Dec 2, 2024
Merged

Feat: 이력서 수정 #37

merged 8 commits into from
Dec 2, 2024

Conversation

Oh-JunTaek
Copy link
Collaborator

#️⃣연관된 이슈

#36

📝작업 내용

이력서 수정 기능.....해냈다!!!!!!!!!!!!!!!!!!!!!!!!!!!
1. dto 설정

httpurl관련 오류가 계속 나서 str로 해도 문제없는거 아닌가? 싶어서 닉에게 물어보니 이미 str로 하고 있다해서 해본 결과 깔끔하게 해결됨! 나도 일민이처럼 이거 이슈해결과정에 적어봐야지 ㅎ
  1. 이력서 수정 프롬프트
  2. api까지 put으로 깔끔하게~~
  3. 스웨거까지 되는거 확인했잖아~~

스크린샷 (선택)

image

💬리뷰 요구사항(선택)

나름 일민이 코드랑 맞춰가면서 할려고 했는데 작업이 원큐에 안끝나서 일단 수정 수정 수정 하다보니 좀 벗어난 것 같아서 리팩토링을 하면서 복습해보려고 함
결론 : 일민아 나 커밋한거 보면서 이렇게 작업했는데 어떤지 한 번 평가해줘 아주 냉철하게

@Oh-JunTaek Oh-JunTaek added the feature added new feature label Nov 29, 2024
@Oh-JunTaek Oh-JunTaek requested a review from IlMinCho November 29, 2024 09:06
@Oh-JunTaek Oh-JunTaek self-assigned this Nov 29, 2024
Copy link
Member

@IlMinCho IlMinCho left a comment

Choose a reason for hiding this comment

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

형 고생하셨어요 전반적으로 훑어봣는데 제가 해논 형식 전부 이해하신거 같아서 보기좋습니다. 멋져요 ㅋㅋㅋㅋ
요청시간이나, 이런거 개선필요한지 체크해보시고 여러 시나리오만들어서 QA하면서 개선할부분들 찾아내면 될거같네요 굿

Comment on lines +283 to +288
"content": (
"You are a friendly and professional resume modification expert."
"Your task is to update only the specified sections of the resume based on the user's request while leaving all other parts unchanged."
"Ensure the modifications are concise, professional, and aligned with the tone of the original resume."
)
Copy link
Member

Choose a reason for hiding this comment

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

프롬프트 작성한건 잘 작성하신거같아요
이 부분은 정확한 정답이 없다보니, 이것저것 일반적인 상황들 넣어서 테스트해보고, 엣지케이스들도 넣어서 테스트하면서
결과값들을 보면서 잘나오는지 확인하는게 방법인거같아요.
저 방식대로 점점 개선해나가면 될겁니다. :)

Comment on lines +1 to +48
from pydantic import BaseModel
from typing import List, Optional

class WorkExperience(BaseModel):
companyName: str
departmentName: str
role: str
workType: str # FULL_TIME, PART_TIME 등
employmentStatus: str # EMPLOYMENT, UNEMPLOYMENT 등
startedAt: str # YYYY-MM 형식
endedAt: Optional[str] # YYYY-MM 형식 또는 빈 값

class ProjectResponse(BaseModel):
projectName: str
projectStartedAt: str # YYYY.MM 형식
projectEndedAt: str # YYYY.MM 형식
skillSet: str
projectDescription: str
repoLink: str
class Link(BaseModel):
linkTitle: str
linkUrl: str

class Education(BaseModel):
schoolType: str # UNIVERSITY_BACHELOR 등
schoolName: str
major: str
graduationStatus: str # ATTENDING, GRADUATED 등
startedAt: str # YYYY-MM 형식
endedAt: Optional[str] # YYYY-MM 형식 또는 빈 값

class Certificate(BaseModel):
certificateName: str
certificateGrade: str
certificatedAt: str # YYYY-MM 형식
certificateOrganization: str

class ResumeResponseDto(BaseModel):
resumeId: str
memberId: int
memberName: str
avatarUrl: str
email: str
position: str
techStack: List[str]
aboutMe: str
tags: Optional[List[str]] # null 가능
workExperiences: List[WorkExperience]
Copy link
Member

Choose a reason for hiding this comment

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

dto도 멋진데요? ㅎㅎ 리턴하고 요청받는것만 약속한대로 유지하고, 나머지는 형이 편한대로 잘짜시면되요 보기에는 문제가 없어보입니다. :) 굿

@Oh-JunTaek Oh-JunTaek merged commit 31701ab into develop Dec 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature added new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants