Skip to content

Conversation

geekrainy
Copy link
Contributor

基于 AntD 二次封装,将 AntD 作为外部依赖,导入类型:

import { Form } from 'antd';
FormProviderProps = React.ComponentProps<Form['Provider']>;

interface ExampleProps {
  onFormChange?: FormProviderProps['onFormChange'];
  onFormFinish?: FormProviderProps['onFormFinish'];
}

构建会报如下错误:

导出函数的返回类型具有或正在使用外部模块“"../node_modules/rc-field-form/lib/FormContext"”中的名称“FormChangeInfo”,但不能为其命名。

此 PR 修复。

@vercel
Copy link

vercel bot commented Mar 22, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/react-component/field-form/1q1zjzn0s
✅ Preview: https://field-form-git-fork-geekrainy-patch-1.react-component.now.sh

@codecov
Copy link

codecov bot commented Mar 22, 2020

Codecov Report

Merging #103 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #103   +/-   ##
=======================================
  Coverage   99.87%   99.87%           
=======================================
  Files          13       13           
  Lines         808      808           
  Branches      174      175    +1     
=======================================
  Hits          807      807           
  Misses          1        1
Impacted Files Coverage Δ
src/FormContext.tsx 100% <ø> (ø) ⬆️

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 8d9f2b9...ac3f60c. Read the comment docs.

@geekrainy
Copy link
Contributor Author

@zombieJ

@zombieJ zombieJ merged commit 6656232 into react-component:master Apr 7, 2020
@zombieJ
Copy link
Member

zombieJ commented Apr 7, 2020

+ rc-field-form@1.1.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants