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

Refactor goods receipts - test task #413

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

BKrasnov
Copy link

@BKrasnov BKrasnov commented Nov 1, 2024

No description provided.

@BKrasnov
Copy link
Author

BKrasnov commented Nov 4, 2024

5h 30m

@BKrasnov
Copy link
Author

BKrasnov commented Nov 5, 2024

  1. Убраны большинство ts-ignore
  2. Раздел src/goods-receipts/store/operations.gr.ts на redux-tookit с небольшим изменением структуры файлов.
  3. Сделан рефакторинг компонента OperationFormRenderer, однако еще очень много чего можно исправить и оптимизировать, например - не нужные useCallback и useMemo, типизация через type, interface, компоненты то стрелочные, то через function, куча лишних константы, которые ток путают и т.п.

Резюме:
На работу потрачено 8 часов. Код - простая форма, задача которой просто собрать данные и отправить на бэк. По хорошему часть филдов должны быть вынеси в сторибук и тогда задача на создание этой фичи упростится только к композиции готовых компонентов с написанием доп. логики. Здесь же основная проблема не в оптимизации формы, а в отсутствии возможности к расширению фичи или функционала.

Поэтому у меня есть следующие вопросы. Благодарю за ответ:

  1. На сколько давно был написан этот код? Предположу, что очень давно, т.к. нет i18n, т.е. компания еще не вышла на зарубежный рынок
  2. На данный момент, есть ли сторибук?
  3. Есть ли архитектура в данном продукте? Собственная или готовые (fsd, atomic)

@BKrasnov BKrasnov changed the title DRAFT: Refactor goods receipts Refactor goods receipts - test task Nov 5, 2024
@cusoksahara
Copy link

Здравствуйте, отвечаю на ваши вопросы.

  1. Код был написан более 5 лет назад
  2. Нет
  3. Да, у нас большой проект, который построен на основе микрофронтенда, где каждый "сервис" может иметь свою собственную архитектуру. Некоторые компоненты, утилиты, хелперы и т.п, которые могут быть использованы независимо - вынесены в отдельный монорепозиторий.

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