feature-sliced / documentation

🍰 Architectural design methodology for Frontend projects
https://feature-sliced.design
MIT License
1.4k stars 146 forks source link

Add new example - Moonlogs #658

Closed pijng closed 3 months ago

pijng commented 3 months ago

Пример с использованием Effector + Forest в качестве встроенного веб-интерфейса в приложение для сбора бизнесовых логов на Go.

https://github.com/pijng/moonlogs

netlify[bot] commented 3 months ago

Deploy Preview for pr-fsd ready!

Name Link
Latest commit 8465e09ffd5c771b7f4aee0c65442ea29b1687d1
Latest deploy log https://app.netlify.com/sites/pr-fsd/deploys/65e892bffeab54000849a4b4
Deploy Preview https://deploy-preview-658--pr-fsd.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

illright commented 3 months ago

Привет, очень интересный проект, спасибо! В целом, к структуре приложения претензий никаких нет, есть только несколько минорных фсдшных моментов:

  1. Немного смущает shared/lib/ui. Не очень понял предназначение этой папки, но мне кажется, такое название вызовет путаницу с shared/ui в вопросах того, куда положить новый кусок кода

  2. Заметил файл pages/index.ts, что запрещено структурой FSD. Имеет ли смысл унести этот файл в app?

  3. На похожей ноте, заметил, что есть нестандартная папка routing. Ее бы тоже предложил сделать сегментом в app, просто для стандартности структуры

Скажи, что думаешь насчёт этого фидбека

pijng commented 3 months ago

Привет. Спасибо за фидбек. По вопросам:

  1. Согласен;
  2. Согласен;
  3. Может, все таки, сами роуты и миддлвары перенести в условный @/shared/routing, а биндинг роутера к хистори и общую инициализацию – в @/app? Просто роуты сейчас импортируются по всему проекту (специфика atomic-router позволяет) и будет, наверное, странно в том же @/shared/ui/sidebar видеть импорт в духе import { homeRoute, membersRoute } from "@/app".
illright commented 3 months ago

Да, справедливое замечание. Имеет смысл разделить роутинг между shared и app

pijng commented 3 months ago

Готово, внес правки: https://github.com/pijng/moonlogs/tree/master/web/src

Убрал shared/lib/ui, перенес инициализацию страниц в app/pages, разделил routing на shared/routing и app.