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

Interface icx_submitdfchtlc RPC #404

Merged
merged 3 commits into from
Jul 6, 2021
Merged

Conversation

surangap
Copy link
Contributor

@surangap surangap commented Jun 21, 2021

What kind of PR is this?:

/kind feature

What this PR does / why we need it:

This PR will interfaceicx_submitdfchtlc RPC to jellyfish

Which issue(s) does this PR fixes?:

Fixes #

Additional comments?:

depends on #402

@defichain-bot defichain-bot added the kind/feature New feature request label Jun 21, 2021
@surangap surangap force-pushed the surangap/icx-submitdfchtlc branch from 92e1b51 to 6cac7de Compare June 21, 2021 19:25
@surangap surangap changed the title Interface icx_submitexthtlc RPC Interface icx_submitdfchtlc RPC Jun 21, 2021
@surangap surangap marked this pull request as ready for review June 22, 2021 06:24
@surangap surangap force-pushed the surangap/icx-listorders branch from 440bb01 to b9a0805 Compare June 23, 2021 16:47
@surangap surangap force-pushed the surangap/icx-submitdfchtlc branch from 6cac7de to 2f6acbd Compare June 23, 2021 16:47
@surangap surangap mentioned this pull request Jun 29, 2021
30 tasks
@surangap surangap force-pushed the surangap/icx-listorders branch from b9a0805 to a698813 Compare July 1, 2021 06:12
@surangap surangap force-pushed the surangap/icx-listorders branch 2 times, most recently from 64723ac to c7e11ac Compare July 2, 2021 10:13
Base automatically changed from surangap/icx-listorders to main July 2, 2021 15:35
@codeclimate
Copy link

codeclimate bot commented Jul 5, 2021

Code Climate has analyzed commit e4dcfc8 and detected 0 issues on this pull request.

View more on Code Climate.

@netlify
Copy link

netlify bot commented Jul 5, 2021

✔️ Deploy Preview for jellyfish-defi ready!

🔨 Explore the source changes: 79a3362

🔍 Inspect the deploy log: https://app.netlify.com/sites/jellyfish-defi/deploys/60e29708d622460008107da9

😎 Browse the preview: https://deploy-preview-404--jellyfish-defi.netlify.app

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
packages/jellyfish/dist/index.umd.js 21.22 KB (+0.67% 🔺) 425 ms (+0.67% 🔺) 183 ms (+3.41% 🔺) 608 ms

@codecov
Copy link

codecov bot commented Jul 6, 2021

Codecov Report

Merging #404 (e4dcfc8) into main (78f2509) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #404      +/-   ##
==========================================
+ Coverage   97.36%   97.40%   +0.04%     
==========================================
  Files          96       98       +2     
  Lines        2841     2893      +52     
  Branches      285      285              
==========================================
+ Hits         2766     2818      +52     
  Misses         75       75              
Impacted Files Coverage Δ
...es/jellyfish-api-core/src/category/icxorderbook.ts 100.00% <100.00%> (ø)
packages/jellyfish-transaction/src/script/index.ts 100.00% <0.00%> (ø)
...ages/jellyfish-transaction/src/script/expansion.ts 100.00% <0.00%> (ø)
...ckages/jellyfish-transaction/src/script/invalid.ts 100.00% <0.00%> (ø)
...ckages/jellyfish-transaction/src/script/mapping.ts 96.73% <0.00%> (+0.07%) ⬆️

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 78f2509...e4dcfc8. Read the comment docs.

@fuxingloh fuxingloh enabled auto-merge (squash) July 6, 2021 05:16
@fuxingloh fuxingloh merged commit e397609 into main Jul 6, 2021
@fuxingloh fuxingloh deleted the surangap/icx-submitdfchtlc branch July 6, 2021 06:28
eli-lim pushed a commit that referenced this pull request Mar 27, 2022
* Added block difficulty to stat endpoint

* Improved code

* Applied changes from code review

* Fixed tests failur]

* Updated code per code review

* Improved code based on code review
canonbrother pushed a commit that referenced this pull request Jun 1, 2022
* Added block difficulty to stat endpoint

* Improved code

* Applied changes from code review

* Fixed tests failur]

* Updated code per code review

* Improved code based on code review
canonbrother pushed a commit that referenced this pull request Jun 1, 2022
* Added block difficulty to stat endpoint

* Improved code

* Applied changes from code review

* Fixed tests failur]

* Updated code per code review

* Improved code based on code review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants