Skip to content
Snippets Groups Projects

Show instance sponsors if OC is set as donation url

Merged Marie requested to merge upd/instancesponsor into develop

What does this PR do?

This was requested by @piuvas on discord after the OC sponsor fetching was added.

All this does is expand the sponsors endpoint to add an instance boolean which is set to true on the instance information page.

It will check if a cached version exists first like the normal sponsor one does for the sharkey page.

If not then it checks the donationUrl is set and if it points to opencollective followed by it fetching all backers.

It then sorts by isActive and if Tier is set on user (otherwise it will show all backers) and then returns and caches it.

Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
  • I have made sure to test this pull request

Merge request reports

Merge request pipeline #1320 passed

Merge request pipeline passed for 4d011e7f

Approved by

Merged by MarieMarie 5 months ago (Oct 8, 2024 6:07pm UTC)

Merge details

  • Changes merged into develop with 72a0f16b.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #1414 passed

Pipeline passed for 72a0f16b on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • dakkar
  • dakkar
  • Marie added 4 commits

    added 4 commits

    Compare with previous version

  • Marie resolved all threads

    resolved all threads

  • Marie requested review from @dakkar

    requested review from @dakkar

  • dakkar
  • dakkar
  • Marie added 1 commit

    added 1 commit

    • 6b459be1 - upd: move totalSponsors and donationUrl check

    Compare with previous version

  • Marie resolved all threads

    resolved all threads

  • Marie added 1 commit

    added 1 commit

    • 9dd51bc9 - upd: fix donationUrl check position

    Compare with previous version

  • Marie resolved all threads

    resolved all threads

  • dakkar
  • Marie added 1 commit

    added 1 commit

    • f08ef28d - upd: fix broken returns and change if statement

    Compare with previous version

  • Marie resolved all threads

    resolved all threads

  • dakkar approved this merge request

    approved this merge request

  • Hazelnoot
  • Hazelnoot
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading