Skip to content

fix: the claim_all_revenue method with unwrap IP option#155

Merged
bonnie57 merged 6 commits intomainfrom
PYSDK-32
Oct 28, 2025
Merged

fix: the claim_all_revenue method with unwrap IP option#155
bonnie57 merged 6 commits intomainfrom
PYSDK-32

Conversation

@bonnie57
Copy link
Contributor

@bonnie57 bonnie57 commented Oct 27, 2025 •
edited
Loading

Description

  1. Fix the claim_all_revenue method with the unwip option
  2. Refactor royalty unit tests and integration tests
  3. Enhance the claim_all_revenue method to reduce the transaction count by using the ipAccount.executeBatch method and
  4. Enhance the _parse_tx_revenue_token_claimed_event method to get data-friendlyly
  5. Import WIP token

Test Plan

Unit tests

image

Integration tests

image

@bonnie57 bonnie57 changed the title fix the claim_all_revenue method with unwip option fix: the claim_all_revenue method with unwip option Oct 27, 2025
@lucas2brh lucas2brh changed the title fix: the claim_all_revenue method with unwip option fix: the claim_all_revenue method with unwrap IP option Oct 28, 2025
@bonnie57 bonnie57 merged commit 9e7aeba into main Oct 28, 2025
8 checks passed
@bonnie57 bonnie57 deleted the PYSDK-32 branch October 28, 2025 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucas2brh lucas2brh lucas2brh approved these changes

@sebsadface sebsadface Awaiting requested review from sebsadface sebsadface is a code owner

@edisonz0718 edisonz0718 Awaiting requested review from edisonz0718 edisonz0718 is a code owner

@AndyBoWu AndyBoWu Awaiting requested review from AndyBoWu AndyBoWu is a code owner

@limengformal limengformal Awaiting requested review from limengformal limengformal is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bonnie57 @lucas2brh