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

feature/typescript tutorial #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

denis-zaichenko
Copy link

After reading these articles, not merge, please, I combine commits.

@denis-zaichenko denis-zaichenko changed the title Feature/typescript tutorial аeature/typescript tutorial Jul 27, 2020
@denis-zaichenko denis-zaichenko changed the title аeature/typescript tutorial feature/typescript tutorial Jul 27, 2020
@@ -0,0 +1,469 @@
# Clear code
Copy link
Member

Choose a reason for hiding this comment

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

This is same as we have in javascript file


Now I am creating some simple form from login to site.

![Form typing](../../assets/clear-code/form-type.png)
Copy link
Member

Choose a reason for hiding this comment

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

It's better to write code instead of using pictures.


![Input fields](../../assets/clear-code/input-field-refactoring.png)

Reading got easier. The name of the component corresponds to the problem it solves. The purpose of the element is evident. There is less code. So we are heading in the right direction!
Copy link
Member

Choose a reason for hiding this comment

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

Decrease amount of words in this file


[Return to Typescript](./README.md).

#### `Partial <T>` - constructs a type with all properties of `T` set to optional.
Copy link
Member

Choose a reason for hiding this comment

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

Decrease explanation here and put this info to all typescript document

4. [Development](docs/development.md)
5. [Tools](docs/tools.md)
6. [Git Flow](docs/git.md)
7. [Clear code](docs/clear-code/README.md)
Copy link
Member

Choose a reason for hiding this comment

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

Better put this info about patterns to react doc with namereact-clean-code

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