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

Implement NCIP-25 #3159

Open
wants to merge 11 commits into
base: development
Choose a base branch
from
Open

Implement NCIP-25 #3159

wants to merge 11 commits into from

Conversation

ipdae
Copy link
Member

@ipdae ipdae commented Feb 10, 2025

@ipdae ipdae force-pushed the feature/issue-3158 branch 2 times, most recently from 6db0d3e to dd17a40 Compare February 11, 2025 13:01
@ipdae ipdae changed the title Feature/issue 3158 Implement NCIP-25 Feb 11, 2025
@ipdae ipdae self-assigned this Feb 11, 2025
@ipdae ipdae force-pushed the feature/issue-3158 branch from dd17a40 to 2b069c3 Compare February 11, 2025 13:08
@ipdae ipdae marked this pull request as ready for review February 13, 2025 04:43
Copy link

cloudflare-workers-and-pages bot commented Feb 18, 2025

Deploying lib9c with  Cloudflare Pages  Cloudflare Pages

Latest commit: f981d38
Status: ✅  Deploy successful!
Preview URL: https://762c2da0.lib9c.pages.dev
Branch Preview URL: https://feature-issue-3158.lib9c.pages.dev

View logs

/// </summary>
[Serializable]
[ActionType(TypeIdentifier)]
public class ClaimWorldBossReward : ActionBase
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

base 타입이 GameAction이여야 하는 것 아닐까요?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

클라이언트의 처리때문이라 전달받았고, GameAction으로 변경해두겠습니다.

@eugene-doobu
Copy link
Member

CalculateTicketPrice 관련 기능 변경 추가 필요해보입니다
-> 월드보스 등수에서 공헌도로 시즌보상이 변경됨에 따라 티켓 구매가격을 1ncg 로 고정


// Retrieve RaiderState
var raiderState = new RaiderState((List)states.GetLegacyState(raiderAddress));
if (raiderState.HasClaimedReward)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

해당 상태 갱신이 안되는 것으로 보입니다

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

클레임했을때 이 값이 변경안된다는거죠? 확인해보겠습니다.

@ipdae
Copy link
Member Author

ipdae commented Feb 18, 2025

@eugene-doobu 티켓가격은 테이블시트에서 조절하는식이라 상태갱신만 반영해뒀습니다.

@ipdae ipdae requested a review from eugene-doobu February 18, 2025 09:29
@eugene-doobu eugene-doobu linked an issue Feb 28, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review
Development

Successfully merging this pull request may close these issues.

fix CalculateContribution 월드보스 공헌도 보상 청구
3 participants