Skip to content

[Job Launcher] Remove routing protocol#3848

Merged
flopez7 merged 7 commits into3806from
flopez/remove-routing-protocol
Apr 7, 2026
Merged

[Job Launcher] Remove routing protocol#3848
flopez7 merged 7 commits into3806from
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 Apr 7, 2026 10:00am
faucet-server Ignored Ignored Preview Apr 7, 2026 10:00am
human-app Skipped Skipped Apr 7, 2026 10:00am
human-dashboard-frontend Skipped Skipped Apr 7, 2026 10:00am
staking-dashboard Skipped Skipped Apr 7, 2026 10:00am

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
11 tasks
Base automatically changed from flopez/3806-1 to 3806 April 7, 2026 09:48
@flopez7 flopez7 removed the do-not-merge PR shouldn't be merged until this label is removed label Apr 7, 2026
@flopez7 flopez7 force-pushed the flopez/remove-routing-protocol branch from 674f4b3 to 923896b Compare April 7, 2026 09:55
@vercel vercel bot temporarily deployed to Preview – human-app April 7, 2026 09:55 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend April 7, 2026 09:55 Inactive
@vercel vercel bot temporarily deployed to Preview – staking-dashboard April 7, 2026 09:55 Inactive
@vercel vercel bot temporarily deployed to Preview – human-app April 7, 2026 10:00 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend April 7, 2026 10:00 Inactive
@vercel vercel bot temporarily deployed to Preview – staking-dashboard April 7, 2026 10:00 Inactive
@flopez7 flopez7 merged commit c0dc909 into 3806 Apr 7, 2026
6 checks passed
@flopez7 flopez7 deleted the flopez/remove-routing-protocol branch April 7, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants