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.

Sign up for GitHub

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

Jump to bottom

Juno @task-5 #1870

Closed
wants to merge 55 commits into from
Closed

Juno @task-5 #1870

wants to merge 55 commits into from

Conversation

PsychoPunkSage
Copy link

NOTE: Please don't merge it. It is an exercise PR

PsychoPunkSage and others added 28 commits May 7, 2024 14:03
@PsychoPunkSage
fix::> Basic str ready
787942a
@PsychoPunkSage
fix::> parsing error reverting back to org
9fe3466
@PsychoPunkSage
fix::> RPC export properly working
8df2f60
@PsychoPunkSage
fix::> block with receipt output
aafed1c
@PsychoPunkSage
fix::> block with txns output
bf778fb
@PsychoPunkSage
fix::> remoaninig push
b0cd907
@PsychoPunkSage
fix::> removed rpc response
a41b802
@PsychoPunkSage
add::> added response file
e09c801
@PsychoPunkSage
fix::> New handler added
301b7b9
@PsychoPunkSage
fix::> Test basic str ready
7b66a6f
@PsychoPunkSage
Merge branch 'NethermindEth:main' into pps-sandbox
07a205f
@PsychoPunkSage
fix::> Test ready
ba7e1b0
@PsychoPunkSage
fix::> Implemented fn to get StateTrie
6380352
@PsychoPunkSage
fix::> Expose NodeFromRoot fn
c67cfd4
@PsychoPunkSage
fix::> impl fn to get nodes from root and a node parser
399b283
@PsychoPunkSage
fix::> impl handler JunoGetNodesFromRoot
d472131
@PsychoPunkSage
fix::> template
fc257c7
@PsychoPunkSage
fix::> Changed the fn name
2e7918b
@PsychoPunkSage
fix::> added new RPC_meth to methods
3fea2be
@PsychoPunkSage
fix::> modified the code
94fc3f8
@PsychoPunkSage
fix::> Handler test written
4a1a516
@PsychoPunkSage
fix::> removed Juno
893d38e
@PsychoPunkSage
fix::> removed unnecessary files
b49f2f5
@PsychoPunkSage
fix::> Test Written
368ae2d
@PsychoPunkSage
fix::> mock_state Node from Root partially implemented
c2ada93
@PsychoPunkSage
fix::> mock_state Node from Root partially implemented 1
c921837
@PsychoPunkSage
fix::> Transfering task 5 code
248ef83
@PsychoPunkSage
Merge branch 'NethermindEth:main' into @intern/task_five
f5a1e6b
@PsychoPunkSage
Copy link
Author

Hi @rianhughes
I have created a fresh PR for task-5. please have a look here.
Thank You

@PsychoPunkSage
Copy link
Author

I tried to implement NodesFromRoot Mock... but I'm facing little difficulty in doing so...
Can you please have a look at the code and tell me what I'm doing Wrong?? Or what changes I have to make??

PsychoPunkSage and others added 21 commits May 28, 2024 18:04
@PsychoPunkSage
fix::> change the error return way
59bcea5
@PsychoPunkSage
fix::> Changed the impl of element
eb55b94
@PsychoPunkSage
Merge branch 'NethermindEth:main' into @intern/task_five
6e6ae5e
@PsychoPunkSage
Merge branch 'NethermindEth:main' into @intern/task_five
547ee7b
@PsychoPunkSage
add::> added response file
d112f04
@PsychoPunkSage
fix::> New handler added
80bfba9
@PsychoPunkSage
fix::> Test basic str ready
4dee1c3
@PsychoPunkSage
fix::> Test ready
be65415
@PsychoPunkSage
fix::> Implemented fn to get StateTrie
c59c985
@PsychoPunkSage
fix::> Expose NodeFromRoot fn
bf4c8e1
@PsychoPunkSage
fix::> impl handler JunoGetNodesFromRoot
65aa53e
@PsychoPunkSage
fix::> template
75f44c2
@PsychoPunkSage
fix::> Changed the fn name
28f5516
@PsychoPunkSage
fix::> added new RPC_meth to methods
26a8a03
@PsychoPunkSage
fix::> modified the code
fabef6a
@PsychoPunkSage
fix::> Handler test written
05e983b
@PsychoPunkSage
fix::> removed Juno
33a6d10
@PsychoPunkSage
fix::> removed unnecessary files
d4603b7
@PsychoPunkSage
fix::> Merge conflict issue
93c0612
@PsychoPunkSage
fix::> Merge conflict II
372c7fd
@PsychoPunkSage
fix::> Copied mock_state.go file
3693d6f
@PsychoPunkSage
Copy link
Author

Hi @rianhughes
I am trying to run the tests but none of them are running...
I'm getting this error
image

@kirugan kirugan added the onboarding Part of onboarding exercise pack label Jul 11, 2024
@kirugan kirugan mentioned this pull request Jul 15, 2024
Closed
@kirugan kirugan closed this Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rianhughes rianhughes rianhughes left review comments

Assignees
No one assigned
Labels
onboarding Part of onboarding exercise pack
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@PsychoPunkSage @rianhughes @kirugan