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

Teku Beacon Node #545

Merged
merged 13 commits into from
Mar 31, 2021
Merged

Teku Beacon Node #545

merged 13 commits into from
Mar 31, 2021

Conversation

BeroBurny
Copy link
Collaborator

Short description of work done

  • implement beacon client picker
  • implement teku beacon node

PR Checklist

  • I have run type check locally
  • I have run linter locally
  • I have run unit and integration tests locally
  • Rebased to master branch / merged master

Changes

  • improve validation
  • refactor beacon client logic (separate API clients for each beacon client)
  • refactor module importing
  • implement beacon client picker

Example

image

Issues

Closes #529

@BeroBurny BeroBurny self-assigned this Mar 25, 2021
@BeroBurny BeroBurny requested a review from a team as a code owner March 25, 2021 11:26
@codecov
Copy link

codecov bot commented Mar 25, 2021

Codecov Report

Merging #545 (705490c) into develop (061ad59) will decrease coverage by 5.93%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #545      +/-   ##
===========================================
- Coverage    58.59%   52.66%   -5.94%     
===========================================
  Files           75       93      +18     
  Lines         1256     1612     +356     
  Branches       151      175      +24     
===========================================
+ Hits           736      849     +113     
- Misses         492      735     +243     
  Partials        28       28              
Flag Coverage Δ
unit 52.66% <50.00%> (-5.94%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/renderer/ducks/validator/slice.ts 13.33% <ø> (ø)
...vices/eth2/client/eth2ApiClient/cgEth2EventsApi.ts 16.12% <0.00%> (ø)
...nderer/services/eth2/client/eth2ApiClient/index.ts 42.85% <ø> (ø)
src/renderer/services/eth2/networks/local.ts 100.00% <ø> (ø)
src/renderer/services/eth2/networks/pyrmont.ts 100.00% <ø> (ø)
src/renderer/services/eth2/client/utils.ts 27.27% <22.22%> (ø)
src/renderer/models/account.ts 66.66% <100.00%> (ø)
.../renderer/services/eth2/client/lighthouse/index.ts 100.00% <100.00%> (ø)
src/renderer/services/eth2/client/module.ts 100.00% <100.00%> (ø)
src/renderer/services/eth2/client/teku/index.ts 100.00% <100.00%> (ø)
... and 18 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 061ad59...705490c. Read the comment docs.

@BeroBurny BeroBurny added the enhancement New feature or request label Mar 25, 2021
@@ -75,6 +80,13 @@ export const ConfigureBeaconNode: React.FunctionComponent<IConfigureBNProps> = (
<h1>Configure Beacon node settings</h1>
<p>You can skip customizing this data if you want to use the default values.</p>

<div className='configure-port'>
Copy link
Member

Choose a reason for hiding this comment

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

tbh, i would prefer clickable client icons instead of dropdown cc @morrigan

src/renderer/ducks/beacon/sagas.ts Outdated Show resolved Hide resolved
src/renderer/ducks/beacon/sagas.ts Outdated Show resolved Hide resolved
# Conflicts:
#	src/renderer/components/ConfigureBeaconNode/ConfigureBeaconNode.tsx
#	src/renderer/ducks/beacon/sagas.ts
#	src/renderer/ducks/validator/sagas.ts
@BeroBurny BeroBurny merged commit 79d6b09 into develop Mar 31, 2021
@BeroBurny BeroBurny deleted the beroburny/teku-beacon branch March 31, 2021 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants