AElfProject / aelf-web3.js

aelf JavaScript SDK
MIT License
16 stars 24 forks source link

fix: 🐛 fix fetch error result and host url #98

Closed potter-sun closed 1 year ago

potter-sun commented 1 year ago

1、Fix HttpProvider.host The last bit cannot contain '/' 2、Modify the interface request return value format

vizipi[bot] commented 1 year ago

Pull request analysis by VIZIPI

Below you will find who is the most qualified team member to review your code. This analysis includes his/her work on the code included in this Pull request, in addition to their experience in code affected by these changes ( partly found within the list of potential missing files below )   Feedback always welcome

Reviewers with knowledge related to these changes

Match % Person Commit Count Common Files
85.71 % 轩辕焮痕 48 6
85.71 % Alef-hz 5 6

Potential missing files from this Pull request

files commonly committed with a subset of this pr, but not committed this time. (click to collapse)
FilePercentilerate
CHANGELOG.md71.43 %5 out of 7 times
yarn.lock71.43 %5 out of 7 times
test/e2e/index.test.js71.43 %5 out of 7 times
src/util/keyStore.js71.43 %5 out of 7 times
jest.node.config.js71.43 %5 out of 7 times
jest.browser.config.js71.43 %5 out of 7 times
build/webpack.node.js71.43 %5 out of 7 times
package-lock.json57.14 %4 out of 7 times
dist/aelf.js.map57.14 %4 out of 7 times
dist/aelf.js57.14 %4 out of 7 times

Committed file ranks

  • 0.00%[examples/browser/requestByFetch.html]
  • 99.03%[dist/aelf.cjs.js]
  • 97.42%[dist/aelf.cjs.js.map]
  • 99.68%[package.json]
  • 99.35%[dist/aelf.umd.js]
  • 90.32%[src/util/httpProvider.js]
  • 98.39%[dist/aelf.umd.js.map]