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] 개인 기록 구현 #214

Merged
merged 28 commits into from
Dec 10, 2024
Merged

[Feat] 개인 기록 구현 #214

merged 28 commits into from
Dec 10, 2024

Conversation

FpRaArNkK
Copy link
Contributor

@FpRaArNkK FpRaArNkK commented Nov 16, 2024

Related Issue

연관된 이슈번호를 작성해주세요
#206

Description

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 개인 일정 -> 기록 화면을 작성했습니다.
    • 화면, 로직, API 연결까지 완료된 상태입니다.
    • 중간에 수정된 기획도 반영되었습니다.

Screenshot (Optional)

Requirements for Review

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

로직은 OrderNumber에 맞춰 모두 구현되어 있으나, 서버에서 리턴하는 OrderNumber가 0으로 고정되어 있습니다. 해당 이슈 해결 후 한 번더 체크해야합니다.
기획, 디자인 부분에서 확정되지 않은 부분들이 여럿 있습니다. 추후 확정되는대로 업데이트 하겠습니다
이미지 뷰어는 별도의 모듈이 필요할 것 같아 별도로 작업하려고 합니다
너무 늦었습니다 죄송합니다 ㅠㅠ

@FpRaArNkK FpRaArNkK added the 💡Feat 새로운 기능 구현 이슈 label Nov 16, 2024
@FpRaArNkK FpRaArNkK requested a review from iHyunWoo November 16, 2024 01:24
@FpRaArNkK FpRaArNkK self-assigned this Nov 16, 2024
Copy link
Contributor

@iHyunWoo iHyunWoo left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

}

/// 이미지 파일들을 S3에 업로드 후, 순서를 맞춰 DiaryImage 배열로 반환합니다.
public func postDiaryImages(scheduleId: Int, images: [UIImage]) async throws -> [DiaryImage] {
Copy link
Contributor

Choose a reason for hiding this comment

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

오... 나중에 참고하겠습니다. 좋네요

Copy link

@Seokki-Kwon Seokki-Kwon left a comment

Choose a reason for hiding this comment

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

확인입니다 고생하셨어유!!

@FpRaArNkK FpRaArNkK merged commit 306d2d8 into develop Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡Feat 새로운 기능 구현 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants