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.

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

[Fix] #100 - 스탬프 등록, 삭제, 수정 실패시 에러 처리 #104

Merged
merged 2 commits into from
Jan 11, 2023
Merged

[Fix] #100 - 스탬프 등록, 삭제, 수정 실패시 에러 처리 #104

merged 2 commits into from
Jan 11, 2023

Conversation

yangsubinn
Copy link
Member

@yangsubinn yangsubinn commented Jan 10, 2023 •
edited
Loading

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

📌 참고 사항

  • 지금은 이미지 용량 엄청 큰 것도 다 잘 올라가고, 압축을 1로 해도 시간만 쪼금 길어질 뿐이구 잘 올라가서 ..
    StampAPI에서 경로나 header 부분 틀리게 해두고 테스트 했습니다
  • 에러 종류를 구체적으로 구분할 수가 없어서, 정상적이지 않은 경우 모두 networkAlert를 보이도록 했습니다
  • post 스크린샷에서 띡 하고 덜 어두워지는 그것은 alertVC의 백그라운드인데 .. 네 .. 🫠

참고자료
https://taehoon95.tistory.com/167
https://icksw.tistory.com/288

📸 스크린샷

기능 스크린샷
post ezgif com-gif-maker - 2023-01-10T221042 114
delete

📮 관련 이슈

@yangsubinn yangsubinn added Fix 문제 해결, 코드 수정 양수🌷 labels Jan 10, 2023
@yangsubinn yangsubinn self-assigned this Jan 10, 2023
L-j-h-c
L-j-h-c approved these changes Jan 11, 2023
View reviewed changes
Copy link
Contributor

@L-j-h-c L-j-h-c left a comment

Choose a reason for hiding this comment

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

멋지네여...나중에는 Error Type을 정의해서 error를 직접 처리하는 쪽으로 해봐도 좋겠네용!

@yangsubinn yangsubinn merged commit fc30863 into sopt-makers:develop Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@L-j-h-c L-j-h-c L-j-h-c approved these changes

@lsj8706 lsj8706 Awaiting requested review from lsj8706

@devxsby devxsby Awaiting requested review from devxsby

Assignees

@yangsubinn yangsubinn

Labels
Fix 문제 해결, 코드 수정 size/M 양수🌷
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Fix] 스탬프 등록시 용량으로 인해 에러 발생하는 경우 처리
2 participants
@yangsubinn @L-j-h-c