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

починил .d.ts #251

Merged
merged 7 commits into from
Jul 16, 2017
Merged

починил .d.ts #251

merged 7 commits into from
Jul 16, 2017

Conversation

slawwan
Copy link
Contributor

@slawwan slawwan commented Jul 15, 2017

структура описания была неверной, сделал правильно, проверил пропсы, добавил опциональности кое-где


export interface IconProps {
color?: string;
name: Array<string>;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

почему массив?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

так добавили, переносил - не заметил, сейчас поправлю, еще в 20px такая же хрень

@nanot1m nanot1m merged commit 0a94b6c into master Jul 16, 2017
@nanot1m nanot1m deleted the dts-fixes branch March 16, 2018 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants