Skip to content

[Job Launcher] Remove routing protocol#3848

Open
flopez7 wants to merge 1 commit intoflopez/3806-1from
flopez/remove-routing-protocol
Open

[Job Launcher] Remove routing protocol#3848
flopez7 wants to merge 1 commit intoflopez/3806-1from
flopez/remove-routing-protocol

Conversation

@flopez7
Copy link
Copy Markdown
Contributor

@flopez7 flopez7 commented Mar 27, 2026

Issue tracking

Freestyle

Context behind the change

Cleanup all routing protocol code since it's not in use.

  • Modified job.dto.ts to make chainId a required field.
  • Removed RoutingProtocolModule and related service as part of refactoring.
  • Updated job.service.ts to utilize Web3Service for oracle selection.
  • Removed routing-protocol interface and service files.
  • Cleaned up constants related to reputation oracles in test files.

How has this been tested?

Deployed and tested locally

Release plan

Check with hCaptcha chainId usage in /job/quick-launch

Potential risks; What to monitor; Rollback plan

None

@flopez7 flopez7 requested a review from portuu3 March 27, 2026 13:27
@flopez7 flopez7 added the do-not-merge PR shouldn't be merged until this label is removed label Mar 27, 2026
@flopez7 flopez7 self-assigned this Mar 27, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 27, 2026

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

5 Skipped Deployments
Project Deployment Actions Updated (UTC)
faucet-frontend Ignored Ignored Preview Mar 27, 2026 3:05pm
faucet-server Ignored Ignored Preview Mar 27, 2026 3:05pm
human-app Skipped Skipped Mar 27, 2026 3:05pm
human-dashboard-frontend Skipped Skipped Mar 27, 2026 3:05pm
staking-dashboard Skipped Skipped Mar 27, 2026 3:05pm

Request Review

@flopez7 flopez7 changed the title Flopez/remove routing protocol [Job Launcher] Remove routing protocol Mar 27, 2026
@flopez7 flopez7 closed this Mar 27, 2026
@flopez7 flopez7 force-pushed the flopez/remove-routing-protocol branch from 4c36610 to 5d7a1db Compare March 27, 2026 14:45
@flopez7 flopez7 reopened this Mar 27, 2026
@flopez7 flopez7 mentioned this pull request Mar 27, 2026
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge PR shouldn't be merged until this label is removed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants