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

Rewrite project to TS #19

Merged
merged 14 commits into from
Jan 23, 2023
Merged

Rewrite project to TS #19

merged 14 commits into from
Jan 23, 2023

Conversation

Wenn911
Copy link
Contributor

@Wenn911 Wenn911 commented Sep 25, 2022

  1. Installed TS libraries
  2. Configurated tsconfig & webpack.config file
  3. Added interfaces
  4. Added types for the component propses
  5. Added types for function parameters and logic

@ashikov
Copy link

ashikov commented Sep 27, 2022

@Wenn911 тут сборка упала

@Wenn911
Copy link
Contributor Author

Wenn911 commented Sep 28, 2022

Поправил, должно работать

@ashikov
Copy link

ashikov commented Sep 29, 2022

@Wenn911 что-то всё равно не собирается

@Wenn911
Copy link
Contributor Author

Wenn911 commented Sep 29, 2022

Так понимаю ошибка с вебпаком связана, подкрутил tsconfig, может сейчас пройдут тесты. Если нет, то буду вебпак смотреть, хотя на мой взгляд там все нормально

@ashikov
Copy link

ashikov commented Sep 30, 2022

@Wenn911 похоже надо смотреть вебпак

@Wenn911
Copy link
Contributor Author

Wenn911 commented Oct 1, 2022

Оказалась очень глупая ошибка из-за невнимательности, запушил, должно пройти

@Wenn911
Copy link
Contributor Author

Wenn911 commented Oct 2, 2022

@ashikov Деплой не проходит, пишет доступа нет

@Wenn911
Copy link
Contributor Author

Wenn911 commented Oct 4, 2022

@dzencot
Copy link
Collaborator

dzencot commented Oct 17, 2022

Пр не забыт. Работа ведётся

@ashikov
Copy link

ashikov commented Nov 18, 2022

@dzencot всё ещё не забыт? :)

@dzencot
Copy link
Collaborator

dzencot commented Dec 12, 2022

@ashikov забыт(
Запланирую на эту неделю работу с этим пр.

@fey
Copy link
Collaborator

fey commented Dec 28, 2022

@dzencot тут все повисло. Какой итог? Ты доделываешь ПР?

@dzencot
Copy link
Collaborator

dzencot commented Jan 9, 2023

@fey к сожалению забыл посмотреть. На этой неделе займусь. Заведу тикет

@dzencot dzencot merged commit dc2a506 into Hexlet:main Jan 23, 2023
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.

4 participants