Skip to content

[API-223] Add v1/users/:userId/purchasers#269

Merged
raymondjacobson merged 1 commit intomainfrom
rj-api-223
Jul 28, 2025
Merged

[API-223] Add v1/users/:userId/purchasers#269
raymondjacobson merged 1 commit intomainfrom
rj-api-223

Conversation

@raymondjacobson
Copy link
Copy Markdown
Member

@raymondjacobson raymondjacobson commented Jul 25, 2025

Bridge doing bridge things
image

@raymondjacobson raymondjacobson requested a review from schottra July 25, 2025 18:29
Copy link
Copy Markdown
Contributor

@schottra schottra left a comment

Choose a reason for hiding this comment

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

Could have sworn I reviewed this already. Sorry for the delay!

WHERE
` + strings.Join(filters, " AND ") + `
ORDER BY
buyer_user_id ASC
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this just arbitrary so that it's deterministic?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yep

@raymondjacobson raymondjacobson merged commit ff4b41b into main Jul 28, 2025
3 checks passed
@raymondjacobson raymondjacobson deleted the rj-api-223 branch July 28, 2025 17:54
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.

2 participants