Conversation
| <> | ||
| {ledgerId && ( | ||
| <tr> | ||
| <td className="blue-link-text"> |
There was a problem hiding this comment.
его точно удалить нужно, просто не похоже на фикс варнинга
| componentWillReceiveProps() { | ||
| this.getAskOrders(); | ||
| this.getBidOrders(); | ||
| } |
There was a problem hiding this comment.
в целом можно прогнать проект через https://github.com/reactjs/react-codemod#rename-unsafe-lifecycles для переименования устаревших методов. Они периодически всречаются
There was a problem hiding this comment.
я согласна, что надо это сделать, но боюсь что может что-то в логике поламаться
Ну и лучше это сделать отдельной таской
There was a problem hiding this comment.
по поводу отдельной таски - это хорошее замечание. Более наглядно и проще что-то поправить, если сломается
| @@ -55,7 +55,9 @@ class MandatoryApproval extends React.Component { | |||
| <div className="modal-box"> | |||
| <form className="modal-form"> | |||
There was a problem hiding this comment.
в данном файле имеются не используемые импорты
| function Stage(number) { | ||
| if (number === "0") { | ||
| return "Registration"; | ||
| } else if (number === "1") { | ||
| return "Processing"; | ||
| } else if (number === "4") { | ||
| return "Expired"; | ||
| } else if (number === "5") { | ||
| return "Done"; | ||
| } | ||
| } |
There was a problem hiding this comment.
switch сюда очень просится)
4 participants
No description provided.