Skip to content

Conversation

@sergeyskachkov-cpu
Copy link
Collaborator

No description provided.

<>
{ledgerId && (
<tr>
<td className="blue-link-text">
Copy link
Collaborator

Choose a reason for hiding this comment

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

его точно удалить нужно, просто не похоже на фикс варнинга

Comment on lines +26 to +29
componentWillReceiveProps() {
this.getAskOrders();
this.getBidOrders();
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

в целом можно прогнать проект через https://github.com/reactjs/react-codemod#rename-unsafe-lifecycles для переименования устаревших методов. Они периодически всречаются

Copy link
Collaborator

Choose a reason for hiding this comment

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

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

по поводу отдельной таски - это хорошее замечание. Более наглядно и проще что-то поправить, если сломается

return (

<div className="modal-box">
<form className="modal-form">
Copy link
Collaborator

Choose a reason for hiding this comment

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

в данном файле имеются не используемые импорты

Comment on lines +7 to 17
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";
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

switch сюда очень просится)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants