Андрей К

1. Первое что открыл:
















Ну тут явно напрашивается какой-то цыкл.

2. Так конечно технически можно...

{
currencyList.map((currencyItem, index) => (

checkedCurrency(currencyItem)}
>{currencyItem}


))
}

Но лучше не писать логику, тот же map в части return. Обычно это выносится в переменную выше, можно в самом методе render, в зависимости от ситуации, а потом уже только поставляется сама переменная в разметку.

3. В глаза бросается

export { default } from './Ticket';

Во-первых, не нужно создавать лишний файл, сама директория уже служит именем компонента, поэтому уже index.js и есть тот самый файл.

Во-вторых, это ещё больше минус, так как создаёт впечатление, что Вы не понимаете как работает import/export.

Если используется default, то не нужно использовать деструктивное присвоение. Нужно просто писать:

export mySuperTicket from './Ticket';

Конечно имя нормальное писать нужно, это в том смысле, что можно любое.

4. Есть ещё моменты, но тут вопрос на какую позицию Вы претендуете. Если джуниор без реального опыта, то не сильно критично как Вы организовываете файлы и папки, так как на работе скажут как нужно, это уже для человека со стажем был бы вопрос, что за фигня.