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

[워크샵] 합성 재료 등록 후 다시 재료 추가 시 기존 등록 재료가 해제되는 현상 #6575

Open
QgamesGeonPyoNoh opened this issue Dec 12, 2024 · 5 comments · Fixed by #6597
Assignees
Labels
bug Something isn't working
Milestone

Comments

@QgamesGeonPyoNoh
Copy link

APV version info

v200260

Client version info

v260.0.0

Device

Windows, Android (9c M), iOS (9c M)

detailed device info

Windows 10, Android 12, iOS 17

Planet

Heimdall

account address info

0x3146FE0E47A1998e14cd8a33640e3648A5A90276

avatar address info

No response

purchase receipt info

No response

What happened?

[내용]
합성 재료 등록 후 다시 재료 추가 시 기존 등록 재료가 해제되는 현상

  • 동일 재료-등급 선택 / 다른 재료-등급 선택 시 모두 동일하게 발생하고 있습니다.
  • Windows, Android, iOS 환경에서 발생 확인했습니다.
default.mp4

How to reproduce the issue

[재현 스텝]

  1. 메인 로비 - 워크샵 - 합성 진입
  2. 합성 가능한 등급의 선택 버튼 클릭
  3. 합성 재료 선택 후 등록
  4. 다시 합성 가능한 등급의 선택 버튼 클릭하여 합성 재료 선택 후 등록
  5. 기존에 등록한 재료가 해제되는 현상 확인

[Should be]
합성 재료 등록 후 재료 추가 시 기존에 등록한 재료가 유지되어야 합니다.

Relevant log output

@QgamesGeonPyoNoh QgamesGeonPyoNoh added the bug Something isn't working label Dec 12, 2024
@QgamesGeonPyoNoh QgamesGeonPyoNoh added this to the v260.0.0 milestone Dec 12, 2024
@eugene-doobu eugene-doobu moved this from Todo to Review in Nine Chronicles Dec 12, 2024
@eugene-doobu
Copy link
Member

현재 1번의 합성에 사용될 재료의 아이템 타입과 등급은 고정되어 있습니다. 따라서 올려주신 영상과 같은 플로우면 기존 등록 재료가 해제되는게 맞습니다.
이와 별개로 기존에 등록되어있는 같은 재료 아이템 타입과 등급도 다시 등록하면 아이템도 동일 플로우를 타서 새로 아이템 등록시 기존 등록된 아이템이 취소 될 것인데, 이 부분을 이슈로 봐야할지 판단 부탁드리겠습니다 @jaeho0103

@jaeho0103
Copy link
Contributor

기존에 재료가 등록되어있는 시점에서 새로운 타입과 등급의 재료를 넣기전에 register 버튼 클릭시 컨펌 메세지 팝업을 띄워줘야 할 것 같습니다. "합성은 한번에 한등급에 대해서만 진행 가능합니다. 기존 선택된 재료를 리셋하고, 새로 집어넣으시겠습니까?" Yes or No 입니다.

@eugene-doobu eugene-doobu moved this from Review to Hold in Nine Chronicles Dec 12, 2024
@eugene-doobu eugene-doobu moved this from Hold to In Progress in Nine Chronicles Dec 12, 2024
@eugene-doobu eugene-doobu linked a pull request Dec 16, 2024 that will close this issue
@eugene-doobu eugene-doobu moved this from In Progress to Review in Nine Chronicles Dec 16, 2024
@eugene-doobu eugene-doobu moved this from Review to QA in Nine Chronicles Dec 16, 2024
@eugene-doobu
Copy link
Member

아래 PR에서 수정하였습니다. 260.0.2에서 확인 부탁드립니다
#6597

@QgamesGeonPyoNoh
Copy link
Author

재료 추가 시 컨펌 메세지 팝업 출력 확인했습니다.
다만, 동일 재료-등급 선택 시에도 해당 팝업이 출력되고 있는데 의도하신 부분인지 문의드립니다.

[테스트 환경]
Windows.v200260-260.0.2
Android.v200260-260.0.2
iOS.v260.0.2(137)

default.mp4

@QgamesGeonPyoNoh QgamesGeonPyoNoh moved this from QA to Todo in Nine Chronicles Dec 16, 2024
@eugene-doobu eugene-doobu moved this from Todo to In Progress in Nine Chronicles Dec 17, 2024
@eugene-doobu
Copy link
Member

위 팝업이 안뜨게 하려면 기존 선택했던 것이랑 같은 타입, 같은 등급 아이템을 선택했을 때 기존에 선택했던 아이템들이 자동선택된 상태로 넘어와야 할 것 같은데 이를 위해선 생각보다 많은 처리가 필요할 것 같습니다. (아이템을 가져오거나 하는 부분에서 기존 다른 코드들에서 사용하던 로직을 가져왔는데 이를 좀 비틀어야 할 것 같습니다) 이와 같은 처리 후순위로 미루어도 괜찮을까요? @jaeho0103

@eugene-doobu eugene-doobu modified the milestones: v260.0.0, v270.0.0 Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

3 participants