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

feat: mobile app #47

Merged
merged 35 commits into from
Feb 19, 2025
Merged

feat: mobile app #47

merged 35 commits into from
Feb 19, 2025

Conversation

aliceinapple
Copy link
Member

@aliceinapple aliceinapple commented Jan 30, 2025

Задача #46

  • собрала базовую структуру проекта
  • добавила заготовки для экранов
  • выяснила, что не могу использовать vanilla/extract для react native, добавила компонент Box с использованием @emotion/native

@aliceinapple aliceinapple self-assigned this Jan 30, 2025
@aliceinapple aliceinapple requested a review from Nelfimov February 5, 2025 16:42
@aliceinapple aliceinapple marked this pull request as ready for review February 5, 2025 16:42
Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

ну и работаем как с монорепой

@aliceinapple aliceinapple requested a review from Nelfimov February 7, 2025 12:22
Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

не используешь монорепу

@aliceinapple
Copy link
Member Author

не используешь монорепу

Ты имеешь ввиду, добавить mobile в workspaces?

@Nelfimov
Copy link
Member

не используешь монорепу

Ты имеешь ввиду, добавить mobile в workspaces?

да и вынести ui и все остальные в отдельные пакеты

@aliceinapple
Copy link
Member Author

не используешь монорепу

Ты имеешь ввиду, добавить mobile в workspaces?

да и вынести ui и все остальные в отдельные пакеты

В mobile я использую node_modules, организовала внутри mobile отдельные пакеты screens и ui, где также использую node_modules. Или лучше вынести пакеты на уровень с mobile? Просто для screens например, логичнее было бы оставить его внутри mobile

@Nelfimov
Copy link
Member

См. как сделаны наши другие репы, примеры и как надо по fsd

@aliceinapple aliceinapple changed the title Feat/mobile app feat: mobile app Feb 12, 2025
Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

поэксперементируй с настройками - возможно удастся запретить yarn из корня удалять node_modules. На крайний случай - postinstall скрипт

@aliceinapple
Copy link
Member Author

поэксперементируй с настройками - возможно удастся запретить yarn из корня удалять node_modules. На крайний случай - postinstall скрипт

@Nelfimov

Эта проблема решается либо добавлением

pnpIgnorePatterns:
  - ./mobile/app/**

в корневой .yarnrc.yml

либо придется добавлять postinstall

@Nelfimov
Copy link
Member

поэксперементируй с настройками - возможно удастся запретить yarn из корня удалять node_modules. На крайний случай - postinstall скрипт

@Nelfimov

Эта проблема решается либо добавлением

pnpIgnorePatterns:
  - ./mobile/app/**

в корневой .yarnrc.yml

Если это не сломает скрипты yarn workspace app ... - то давай

@aliceinapple
Copy link
Member Author

@Nelfimov

  • добавила pnpIgnorePatterns
  • также нужно пофиксить тайпчек и линтер, могу начать фикс в рамках этого пр-а?

@Nelfimov
Copy link
Member

@Nelfimov

  • добавила pnpIgnorePatterns

  • также нужно пофиксить тайпчек и линтер, могу начать фикс в рамках этого пр-а?

Отдельно

Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

если у тебя ui находится в mobile, то ему не нужен префикс mobile. Либо убирай префикс либо выноси на уровень корня

@aliceinapple
Copy link
Member Author

если у тебя ui находится в mobile, то ему не нужен префикс mobile. Либо убирай префикс либо выноси на уровень корня

так у меня два ui, для mobile отдельный и отдельный для app

@Nelfimov
Copy link
Member

если у тебя ui находится в mobile, то ему не нужен префикс mobile. Либо убирай префикс либо выноси на уровень корня

так у меня два ui, для mobile отдельный и отдельный для app

знаю, но держать отдельный mobile-ui когда мы обычно это делаем в глобале - некорректно.

@aliceinapple
Copy link
Member Author

если у тебя ui находится в mobile, то ему не нужен префикс mobile. Либо убирай префикс либо выноси на уровень корня

Переместила ui-mobile на уровень корня, т.к., если переименовать на ui, при запуске yarn install из корня, возникает проблема с дублирующимися именами воркспейсов

@Nelfimov Nelfimov merged commit 59363ba into master Feb 19, 2025
4 of 6 checks passed
@Nelfimov Nelfimov deleted the feat/mobile-app branch February 19, 2025 07:22
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