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

Update Grinding: add materials #2688

Open
U-lis opened this issue Jul 16, 2024 · 4 comments · Fixed by #2735
Open

Update Grinding: add materials #2688

U-lis opened this issue Jul 16, 2024 · 4 comments · Fixed by #2735
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@U-lis
Copy link
Contributor

U-lis commented Jul 16, 2024

기획서(노션) : https://www.notion.so/planetarium/9c70cecb25814c9ab6a7d1612d0d507d

@U-lis U-lis self-assigned this Jul 16, 2024
@U-lis U-lis converted this from a draft issue Jul 16, 2024
@U-lis U-lis added the enhancement New feature or request label Jul 16, 2024
@U-lis U-lis added this to the v230.0.0 milestone Jul 16, 2024
@tyrosine1153 tyrosine1153 assigned tyrosine1153 and unassigned U-lis Aug 5, 2024
@tyrosine1153
Copy link
Contributor

현님이 급한 다른 이슈 대응으로 이 이슈 담당이 어려워지셔서 클라이언트 대응하던 제가 lib9c 작업도 같이 대응하기로 했습니다.

@tyrosine1153
Copy link
Contributor

tyrosine1153 commented Aug 5, 2024

그라인드 재료 추가 관련 시트, 액션 수정 (예상 일정 : 2일, 8/6~7)

  • 기획 리뷰
    • 슬랙 스레드 또는 구두로 기획에 대한 질문을 재호님께 드리기
  • 시트 수정
  • 액션 수정
    • 시트를 참조해 Material item을 생성하고 인벤토리에 넣어주도록... 수정
    • 결과 연산은 Helper 클래스에 메서드로 따로 빼서 클라이언트에서 공유하도록

이외 챙길 것

  • 재료가 거래불가 속성이 되도록 하는데 신경써야할 부분이 있는지 확인하기
  • 최근에 시트, 액션 수정한 참고하기 좋은 pr 있는지 찾아보기

@tyrosine1153
Copy link
Contributor

tyrosine1153 commented Aug 5, 2024

현님께서 이전에 BE 작업 관련 문서 작성해주신 부분을 전달받았습니다.
https://www.notion.so/planetarium/BE-lib9c-9c72bebc32f64088a73723d1e0501ef0?pvs=4#7994c82e1d814bf1a6ffc0a87abfd8fc

memo

  • 테스트 코드도 챙기는 것 잊지 말기
  • DP 관련 대응은 현님께서 커스텀 장비 하면서 같이 해주시기로 함

@tyrosine1153
Copy link
Contributor

재호님께 기획 리뷰 및 질문에 대한 답변 받았습니다.
https://planetariumhq.slack.com/archives/C07CDFRGV8S/p1722845815262209

memo

  • 컬럼은 예상했던 대로 (id, count) * 2 총 네 개 추가하는 식으로 가도 될 듯
  • 크게 신경쓸 부분 없음 (커스텀 장비에 대해 따로 신경쓰거나, 레벨업, 스테이킹 처리를 고민할 부분)
  • 그라인딩 결과로 보여주는 재료 아이템은 인벤토리 재료 탭의 정렬을 따름(이후 커스텀 제작에서 추가 예정인 아이템 포함)
    • 크리스탈은 필수이기 때문에 필수로 활성화, 이후의 재료에 정렬을 적용...
  • 다음주 tf 주간까지 포함되면 good, 이번주 금요일 싱글 빌드에 나오면 best (이를 위해 선머지, 후리뷰도 고려할 수 있음)
  • 시트, 액션, 테스트 케이스를 동시 확정하는 것보다 클라이언트 UI 구조 확정, lib9c 시트/액션에 대한 선리뷰를 모두 받고 테스트 케이스를 세우는게 효율적일 수 있겠음.

@tyrosine1153 tyrosine1153 moved this from Todo to In Progress in Nine Chronicles Aug 6, 2024
@tyrosine1153 tyrosine1153 moved this from Review to Hold in Nine Chronicles Aug 8, 2024
@tyrosine1153 tyrosine1153 moved this from Hold to Review in Nine Chronicles Aug 8, 2024
@tyrosine1153 tyrosine1153 moved this from Review to Merged in Nine Chronicles Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants