-
Notifications
You must be signed in to change notification settings - Fork 45
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
Feature/additional pet #2871
Feature/additional pet #2871
Conversation
이외에 다른 작업은 하지 않아도 pet 소환 + 강화 + 장비제작 등등 전부 다 잘 작동하나요? |
@@ -34,125 +33,7 @@ public AdditionalOptionRateByFixedValueTest() | |||
{ | |||
var sheets = TableSheetsImporter.ImportSheets(); | |||
sheets[nameof(PetOptionSheet)] = @"ID,_PET NAME,PetLevel,OptionType,OptionValue | |||
1001,D:CC 블랙캣,1,ReduceRequiredBlock,5.5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
시트데이터가 변경될 경우에 영향을 안받도록 의도적으로 시트 데이터를 고정시켜둔것처럼 보이는데, 괜찮은 변경일까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
일단 펜리르를 제외하고는 사용하지 않는 데이터입니다. 굳이 시트 데이터를 왜 갖고와서 사용했는지 보니까, 펜리르의 펫 옵션인 AdditionalOptionRate~를 테스트 하려고 했는데 랜덤성이 있거나 하는거 때문인지 AdditionalOptionRateByFixedValue라는 펫 옵션 타입을 추가해서 테스트한 것으로 보였습니다. 위 상황에서 굳이 다른 데이터는 들고 있을 필요 없다고 생각해서 테스트에 사용하는 데이터 말고 제거하는 것으로 결정했습니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🆗
추가 펫이 생길걸 가정하고 데이터를 추가합니다