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

이슈 수정화면 개선 #248

Merged
merged 3 commits into from
Nov 10, 2020
Merged

이슈 수정화면 개선 #248

merged 3 commits into from
Nov 10, 2020

Conversation

sihyungyou
Copy link
Collaborator

@sihyungyou sihyungyou commented Nov 10, 2020

Issue Number

Close #126

변경사항

이슈 수정화면에서 이슈번호가 보이도록 수정
이슈 수정화면으로 넘기는 데이터 구조체로 변경

새로운 기능

TODO
테스트코드 작성 후 코드 리팩토링

작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

체크리스트

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?
  • 새로운 테스트와 기존의 테스트가 변경사항에 대해 만족하는가?

- previousData 구조체로 정의하여 title/description/issueNumber를 전달
- 이슈 수정 시 같은 화면이지만 새이슈/댓글추가 대신 이슈번호가 보이도록 변경
@sihyungyou sihyungyou added iOS 🍎 iOS 0_Merge: Dev-iOS Dev-iOS 브랜치로 Merge 합니다. labels Nov 10, 2020
@sihyungyou sihyungyou added this to the iOS-Week3 milestone Nov 10, 2020
@sihyungyou sihyungyou requested a review from SHIVVVPP November 10, 2020 12:52
@sihyungyou sihyungyou self-assigned this Nov 10, 2020
@SHIVVVPP
Copy link
Collaborator

마스터로 머지하셨어요!

@sihyungyou sihyungyou changed the base branch from master to dev-ios November 10, 2020 12:55
@sihyungyou sihyungyou merged commit cd44515 into boostcamp-2020:dev-ios Nov 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0_Merge: Dev-iOS Dev-iOS 브랜치로 Merge 합니다. iOS 🍎 iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

새 이슈 등록/이슈 수정 화면 구현
2 participants