Skip to content

Correct validator exit website URL #7373

Closed
dmattingley wants to merge 2 commits intosigp:unstablefrom
dmattingley:patch-1
Closed

Correct validator exit website URL #7373
dmattingley wants to merge 2 commits intosigp:unstablefrom
dmattingley:patch-1

Conversation

@dmattingley
Copy link

During a voluntary validator exit, the cli instructs the user to visit a website to obtain a secret phrase, the current url returns an Access Denied error:
https://lighthouse-book.sigmaprime.io/voluntary-exit.html

The correct url, that provides information and the secret phrase is:
https://lighthouse-book.sigmaprime.io/validator_voluntary_exit.html

So this PR updates WEBSITE_URL in account_manager/src/validator/exit.rs to reflect that.

Proposed Changes

Update voluntary exit url from https://lighthouse-book.sigmaprime.io/voluntary-exit.html to https://lighthouse-book.sigmaprime.io/validator_voluntary_exit.html

Additional Info

No additional information required.

@CLAassistant
Copy link

CLAassistant commented Apr 29, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@jimmygchen jimmygchen left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

Copy link
Member

@jimmygchen jimmygchen left a comment

Choose a reason for hiding this comment

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

Could you update this PR to target unstable please?

@jimmygchen jimmygchen added waiting-on-author The reviewer has suggested changes and awaits thier implementation. docs Documentation labels Apr 29, 2025
@dmattingley dmattingley changed the base branch from stable to unstable April 29, 2025 07:44
@dmattingley
Copy link
Author

Could you update this PR to target unstable please?

That's been updated to unstable now, @jimmygchen

Copy link
Member

@jimmygchen jimmygchen left a comment

Choose a reason for hiding this comment

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

Thanks!

@jimmygchen jimmygchen added ready-for-merge This PR is ready to merge. and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Apr 30, 2025
@michaelsproul michaelsproul removed the ready-for-merge This PR is ready to merge. label Apr 30, 2025
@michaelsproul
Copy link
Member

Already fixed on unstable.

@michaelsproul
Copy link
Member

We could have tried to include this for v7.0.1, but the merge window has already passed. We are about to release v7.0.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants