Skip to content

feat(reviewer): add daily lb rank multipliers#531

Merged
dhamariT merged 7 commits into
hackclub:mainfrom
ObayM:sw-payouts-multi
Jun 22, 2026
Lines changed: 123 additions & 25 deletionsions
Merged

feat(reviewer): add daily lb rank multipliers#531
dhamariT merged 7 commits into
hackclub:mainfrom
ObayM:sw-payouts-multi

Conversation

@ObayM

@ObayM ObayM commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

what's this do?

Reviewers now earn bonus stardust based on their daily leaderboard rank :)

show it works

image

ai?

Claude

@ObayM ObayM marked this pull request as draft June 12, 2026 19:51
ObayM added ed 2 commits June 15, 2026 21:28
@ObayM ObayM marked this pull request as ready for review June 15, 2026 18:39
db/schema.rb Outdated
t.integer "adjusted_amount"
t.bigint "admin_id"
t.integer "amount", null: false
t.integer "amount"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

yo this is the blocker for me. schema.rb makes amount nullable here and drops the user_id_pending unique index right below, but the only migration in this PR is the stardust float one. that unique index is the whole thing stopping a user from opening 2 pending payouts at once. either ur missing a migration or schema.rb got regenerated off another branch. cant merge til this is reconciled

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, my bad. Thanks for pointing it out, it’s been fixed now :)

ObayM added ed 3 commits June 21, 2026 00:43
…per, fix hours_at_ship typo
…dex on reviewer_payout_requests

These were lost during the merge auto-resolution. The partial unique index
(WHERE aasm_state = 'pending') is the DB-level guard that prevents a user
from opening two pending payout requests simultaneously.sly.
@ObayM ObayM requested a review from rom dhamariT June 20, 2026 21:56
@dhamariT dhamariT added this pull request to the he merge queue Jun 22, 2026
Merged via the queue into hackclub:main with commit t b1ce879 Jun 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

None yet

2 participants