Skip to content

fix: update delegator migration form contract interactions#382

Merged
Jipperism merged 3 commits intomainfrom
fix/313-Fix-Delegator-Migration-Widget-for-Returning-Delegators
Nov 19, 2025
Merged

fix: update delegator migration form contract interactions#382
Jipperism merged 3 commits intomainfrom
fix/313-Fix-Delegator-Migration-Widget-for-Returning-Delegators

Conversation

@Jipperism
Copy link
Collaborator

@Jipperism Jipperism commented Nov 14, 2025

  • Updated package.json to replace Material-UI with MUI components.
  • Adjusted imports in various components (ThreeBoxSteps, broadcaster, orchestrator, delegator) to use MUI.
  • Modified stepper styles to align with MUI's new class names.
  • Enhanced the migration logic in the delegator component to utilize updated contract interactions and improved error handling.

@vercel
Copy link
Contributor

vercel bot commented Nov 14, 2025 •
edited
Loading

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

Project Deployment Preview Comments Updated (UTC)
explorer-arbitrum-one Ready Ready Preview Comment Nov 19, 2025 11:17am

@Jipperism
Copy link
Collaborator Author

Jipperism commented Nov 14, 2025

@rickstaa please have a look at the update of the contract interaction code in https://github.com/livepeer/explorer/pull/382/files#diff-2692a8c75c6a7872043b3a99fcae949f84621cc634b5628d476a9223472d5d90L268 as well as the preview deployment of the form at https://explorer-arbitrum-one-git-fix-313-fi-5b0bae-livepeer-foundation.vercel.app/migrate/delegator. If you agree with the implementation I'll update the other two migration forms as well.

@Jipperism Jipperism requested a review from rickstaa November 14, 2025 11:04
@Jipperism Jipperism self-assigned this Nov 14, 2025
@Jipperism Jipperism linked an issue Nov 14, 2025 that may be closed by this pull request
Fix Delegator Migration Widget for Returning Delegators #313
Closed
rickstaa
rickstaa approved these changes Nov 17, 2025
View reviewed changes
Copy link
Member

@rickstaa rickstaa left a comment

Choose a reason for hiding this comment

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

@Jipperism works for me however it looks like the other widgets also are broken see #389 feel free to merge this later.

@github-project-automation github-project-automation bot moved this from Backlog to Todo in Explorer Revamp Nov 17, 2025
@Jipperism Jipperism force-pushed the fix/313-Fix-Delegator-Migration-Widget-for-Returning-Delegators branch from 7cb1b1f to a570412 Compare November 19, 2025 10:22
@Jipperism
fix: update delegator migration form contract interactions
76126eb 
- Updated package.json to replace Material-UI with MUI components.
- Adjusted imports in various components (ThreeBoxSteps, broadcaster, orchestrator, delegator) to use MUI.
- Modified stepper styles to align with MUI's new class names.
- Enhanced the migration logic in the delegator component to utilize updated contract interactions and improved error handling.
@Jipperism Jipperism force-pushed the fix/313-Fix-Delegator-Migration-Widget-for-Returning-Delegators branch 2 times, most recently from 9f10884 to b006de6 Compare November 19, 2025 11:01
@Jipperism Jipperism force-pushed the fix/313-Fix-Delegator-Migration-Widget-for-Returning-Delegators branch from b006de6 to 4edfbe9 Compare November 19, 2025 11:11
@Jipperism Jipperism marked this pull request as ready for review November 19, 2025 11:12
@Jipperism Jipperism force-pushed the fix/313-Fix-Delegator-Migration-Widget-for-Returning-Delegators branch from 4edfbe9 to ac1b3f5 Compare November 19, 2025 11:13
@Jipperism Jipperism force-pushed the fix/313-Fix-Delegator-Migration-Widget-for-Returning-Delegators branch from ac1b3f5 to c0857a5 Compare November 19, 2025 11:14
@Jipperism
Copy link
Collaborator Author

Jipperism commented Nov 19, 2025

@rickstaa @ECWireless updated the PR to also include fixes to broadcaster and orchestrator forms. Should be ready to go now.

@Jipperism Jipperism requested a review from rickstaa November 19, 2025 11:14
rickstaa
rickstaa approved these changes Nov 19, 2025
View reviewed changes
Copy link
Member

@rickstaa rickstaa left a comment

Choose a reason for hiding this comment

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

LGTM now thanks!

@rickstaa rickstaa mentioned this pull request Nov 19, 2025
Merged
@Jipperism Jipperism merged commit 25e2df7 into main Nov 19, 2025
2 checks passed
@Jipperism Jipperism deleted the fix/313-Fix-Delegator-Migration-Widget-for-Returning-Delegators branch November 19, 2025 13:32
@github-project-automation github-project-automation bot moved this from Todo to Done in Explorer Revamp Nov 19, 2025
@rickstaa rickstaa mentioned this pull request Nov 20, 2025
Closed
@rickstaa rickstaa mentioned this pull request Mar 17, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rickstaa rickstaa rickstaa approved these changes

@ECWireless ECWireless Awaiting requested review from ECWireless ECWireless is a code owner

Assignees

@Jipperism Jipperism

Labels

None yet

Projects

No open projects
Explorer Revamp
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix Delegator Migration Widget for Returning Delegators

2 participants

@Jipperism @rickstaa