Skip to content

Conversation

@shiyasmohd
Copy link
Contributor

Changes made in this PR

  • Add /relayers/:chain_id/send_random endpoint.
  • Extract shared send_transaction logic.
  • Filter out paused and internal-only relayers from random selection

Closes #22

@vercel
Copy link

vercel bot commented Oct 6, 2025

@shiyasmohd is attempting to deploy a commit to the joshaavecom's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Owner

@joshstevens19 joshstevens19 left a comment

Choose a reason for hiding this comment

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

thanks for the feature added a list of things we need to do to get this mergeable shout if you have any questions

@shiyasmohd
Copy link
Contributor Author

thanks for the review @joshstevens19 . Working on the requested changes.

@joshstevens19
Copy link
Owner

thanks for the review @joshstevens19 . Working on the requested changes.

no worries thanks or the PR - also FYI clippy issues need fixing as well - https://github.com/joshstevens19/rrelayer/actions/runs/18270302259/job/52023412499?pr=28

@shiyasmohd
Copy link
Contributor Author

@joshstevens19 PR is ready for review now.

Copy link
Owner

@joshstevens19 joshstevens19 left a comment

Choose a reason for hiding this comment

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

hey gave another review just a few last changes required thanks

@vercel
Copy link

vercel bot commented Oct 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
rrelayer Ready Ready Preview Comment Oct 12, 2025 1:33pm

@shiyasmohd shiyasmohd force-pushed the shiyasmohd/select-random-relayer branch from 4a849d1 to 27ec449 Compare October 8, 2025 03:38
@shiyasmohd
Copy link
Contributor Author

@joshstevens19 Thanks for the review. I've pushed the changes.

Copy link
Owner

@joshstevens19 joshstevens19 left a comment

Choose a reason for hiding this comment

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

looks great thanks for the PR

@joshstevens19
Copy link
Owner

Just need to run some tests on this tomorrow then we can merge thanks again

@shiyasmohd
Copy link
Contributor Author

Sure!

@joshstevens19
Copy link
Owner

Did some fixes to this but I can probably get this live tomorrow now @shiyasmohd

@joshstevens19 joshstevens19 merged commit c5c0c18 into joshstevens19:master Oct 21, 2025
12 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.

Send transaction and make it pick a random relayer

2 participants