search

MeteoraAg / dlmm-sdk

An SDK for building applications on top of Dynamic CLMM
67 stars 57 forks source link

Fix empty reward claims #87

Closed shadeglare closed 5 months ago

shadeglare commented 5 months ago

This patch excludes positions without any fees and/or rewards from the claimAllRewards method.

shadeglare commented 5 months ago

Looks good! Do you mind bump the sdk version in package.json & update the changelog?

Bumped the sdk version to 1.0.48. Where should I put the changelog? Thanks.

00xSam commented 5 months ago

Looks good! Do you mind bump the sdk version in package.json & update the changelog?

Bumped the sdk version to 1.0.48. Where should I put the changelog? Thanks.

https://github.com/MeteoraAg/dlmm-sdk/blob/main/CHANGELOG.md

shadeglare commented 5 months ago

Looks good! Do you mind bump the sdk version in package.json & update the changelog?

Bumped the sdk version to 1.0.48. Where should I put the changelog? Thanks.

https://github.com/MeteoraAg/dlmm-sdk/blob/main/CHANGELOG.md

Done.

00xSam commented 5 months ago

There are conflicts

shadeglare commented 5 months ago

There are conflicts

Resolved as of now.

00xSam commented 5 months ago

@shadeglare I've published @meteora-ag/dlmm@1.0.48, thanks for your contributions