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

QuestView와 QuestCell 완성 #9

Merged
merged 2 commits into from
Nov 15, 2022
Merged

QuestView와 QuestCell 완성 #9

merged 2 commits into from
Nov 15, 2022

Conversation

sprituz
Copy link
Member

@sprituz sprituz commented Nov 14, 2022

📕 Issue Number

Close #5

📙 작업 내역

구현 내용 및 작업 했던 내역

  • QuestView 생성

image

  • QuestCell 생성

image

📘 작업 유형

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

📋 체크리스트

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

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • constraint 가 제대로 된질 모르겠습니다.
  • 만들긴 했지만 직접 테이블뷰에 적용해봐야 제대로 되는지 알 수 있을것 같습니다.
  • 충돌 위험이 있기에 asset은 추가하지 않고 기존에 있던 색상을 이용했습니다 (추후에 수정해야함)



@sprituz sprituz changed the title [feat] QuestView와 QuestCell 완성 QuestView와 QuestCell 완성 Nov 14, 2022
@jinwoong16 jinwoong16 merged commit 90002f9 into develop Nov 15, 2022
@jinwoong16 jinwoong16 linked an issue Nov 15, 2022 that may be closed by this pull request
2 tasks
@sprituz sprituz deleted the feature/QuestCell branch November 16, 2022 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quest cell
2 participants