Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fee Recipient : ux log fixes and prevent validator from calling beacon node if flags are not provided. #10406

Merged
merged 2 commits into from
Mar 22, 2022

Conversation

james-prysm
Copy link
Contributor

@james-prysm james-prysm commented Mar 22, 2022

What type of PR is this?
Bug fix

relates to #10404
relates to #10292
What does this PR do? Why is it needed?
This pr improves the user experience of logs for fee recipients as well as prevents the validator client from sending anything to the beacon node if no flags are used from the validator client. The beacon node will continue to run with its own fee recipient values.

@james-prysm james-prysm added Bug Something isn't working Cleanup Code health! Merge PRs related to the great milestone the merge labels Mar 22, 2022
@james-prysm james-prysm self-assigned this Mar 22, 2022
@james-prysm james-prysm requested a review from a team as a code owner March 22, 2022 16:30
@james-prysm james-prysm changed the title initial commit Fee Recipient : ux log fixes and prevent validator from calling beacon node if flags are not provided. Mar 22, 2022
@james-prysm james-prysm merged commit 601987f into develop Mar 22, 2022
@delete-merged-branch delete-merged-branch bot deleted the fee-recipient-log-ux-fix branch March 22, 2022 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Cleanup Code health! Merge PRs related to the great milestone the merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants