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

OpenCollective should use monthly amount instead of total #28

Closed
5 tasks done
andreynering opened this issue Dec 5, 2022 · 2 comments · Fixed by #35
Closed
5 tasks done

OpenCollective should use monthly amount instead of total #28

andreynering opened this issue Dec 5, 2022 · 2 comments · Fixed by #35

Comments

@andreynering
Copy link

Describe the bug

I just tried this by adding both GitHub Sponsors and Open Collective tokens. The way this tool handle each is different.

For GitHub, it's taking into account the monthly value (as it should), but for Open Collective it's using the total, which results is a wrong output image.

Reproduction

Use GitHub and OpenCollective at the same time and see that they use different values.

System Info

System:
    OS: macOS 12.6
    CPU: (10) arm64 Apple M1 Pro
    Memory: 89.61 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.13.1 - ~/.nvm/versions/node/v16.13.1/bin/node
    Yarn: 1.22.18 - ~/.nvm/versions/node/v16.13.1/bin/yarn
    npm: 8.6.0 - ~/.nvm/versions/node/v16.13.1/bin/npm
  Browsers:
    Brave Browser: 101.1.38.115
    Chrome: 108.0.5359.94
    Firefox: 107.0.1
    Firefox Developer Edition: 108.0
    Safari: 16.0

Used Package Manager

yarn

Validations

  • Follow our Code of Conduct
  • Read the Contributing Guide.
  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
  • The provided reproduction is a minimal reproducible of the bug.
@antfu
Copy link
Member

antfu commented Dec 19, 2022

I don't use OC much and don't have enough data to test. It would be great if you could identify the issue and send a PR. Thanks.

@andreynering
Copy link
Author

I tried the fix the it's still considering the all time donated value instead of monthly.

Also, sponsorkit is unable to detect past sponsors from OpenCollective and show then as active sponsors.

Sorry, I'm trying to be demanding, I'm just reporting a bug 🙂 I'll consider using this for GitHub Sponsors only for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants