Skip to content
This repository has been archived by the owner on Mar 2, 2022. It is now read-only.

feat(useFetchData): 增加错误抛出 #636

Merged
merged 1 commit into from
Jul 19, 2020

Conversation

zhijs
Copy link
Contributor

@zhijs zhijs commented Jul 17, 2020

onRequestError 这个方法不是必传的参数,在没有传递这个参数的时候,如果在请求数据和数据处理过程中,出现问题,比如调用了错误的方法,错误会被吞并,导致难以定位问题。

@codecov-commenter
Copy link

Codecov Report

Merging #636 into master will decrease coverage by 0.24%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #636      +/-   ##
==========================================
- Coverage   17.43%   17.19%   -0.25%     
==========================================
  Files          21       21              
  Lines        1147     1146       -1     
  Branches      374      374              
==========================================
- Hits          200      197       -3     
- Misses        947      949       +2     
Impacted Files Coverage Δ
src/Table.tsx 20.47% <ø> (-0.28%) ⬇️
src/component/toolBar/index.tsx 6.77% <0.00%> (-0.12%) ⬇️
src/useFetchData.tsx 31.25% <40.00%> (-2.60%) ⬇️

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 11c3709...e097315. Read the comment docs.

@chenshuai2144 chenshuai2144 merged commit 04779b0 into ant-design:master Jul 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants