Skip to content

Conversation

@illright
Copy link
Member

Background

Changelog

@netlify
Copy link

netlify bot commented Apr 28, 2023

Deploy Preview for pr-fsd ready!

Name Link
🔨 Latest commit a68d561
🔍 Latest deploy log https://app.netlify.com/sites/pr-fsd/deploys/644b7a17640f5d00081ccabd
😎 Deploy Preview https://deploy-preview-574--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 settings.

@azinit azinit changed the base branch from master to current April 28, 2023 09:04
@netlify
Copy link

netlify bot commented Apr 30, 2023

Deploy Preview for current-feature-sliced ready!

Name Link
🔨 Latest commit 536967b
🔍 Latest deploy log https://app.netlify.com/sites/current-feature-sliced/deploys/644eb985eac11600084b9745
😎 Deploy Preview https://deploy-preview-574--current-feature-sliced.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 settings.

Copy link
Member

Choose a reason for hiding this comment

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

nitpick: Очепятка походу)

- pages/edit-node/ui/
+ pages/edit-note/ui/

Copy link
Member Author

Choose a reason for hiding this comment

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

Не понял

Copy link
Member

Choose a reason for hiding this comment

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

Ну фичи и сущности работают с note (заметкой), а страница почему-то с node (с какой-то нодой, цепочкой)

Ощущение что это опечатка
image
image

Copy link
Member Author

Choose a reason for hiding this comment

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

а, блин, теперь понял)) да, это опечатка

Copy link
Member

Choose a reason for hiding this comment

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

praise: Сам туториал пушка, тот самый визуальный которого не хватало!

Copy link
Member

@azinit azinit May 1, 2023

Choose a reason for hiding this comment

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

thoughts: Но все еще главная проблема при использовании FSD - это умение умеренно и постепенно затаскивать методологию к себе в голову и проект

Т.е. люди просто берут к себе в проект слой entities/features, даже не понимая зачем им это особо нужно
(даже если в соло, даже если в доке напишем - пока они сами не настрадаются настолько, сколько нужно чтоб понять необходимость фич - им выделение фич не будет давать пользы - см. также #pain_driven_development)

Говорю это опираясь на наши внутренние дискусии кортимы, консультации и вопросы в чате

image

Copy link
Member

@azinit azinit May 1, 2023

Choose a reason for hiding this comment

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

suggestion: Получится ли в этот распорядок туториала (и твой план в целом) добавить визуально эту идею ниже?

  1. сначала затаскиваются в проект app pages widgets shared
    image
  2. потом entities
    image
  3. и в конце уже features
    image

Copy link
Member

Choose a reason for hiding this comment

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

@illright вот это главное глянь пож)

Copy link
Member

Choose a reason for hiding this comment

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

не хочется чтоб ломало твой план по туториалу, но мб будут мысли как эту идею учесть в туториале
(чтоб люди опять не задавались вопросом что есть фича и зачем нужна фича, когда есть виджеты)

Copy link
Member Author

Choose a reason for hiding this comment

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

да, это самый сложный коммент просто, про него отдельно подумаю

Copy link
Member

@azinit azinit May 1, 2023

Choose a reason for hiding this comment

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

Возможно твой туториал стоит оставить как есть, но сделать другой туториал/гайд где именно про этапы затаскивания говорится (желательно тоже в get-started чтоб лежало и было перед глазами)

UPD: Т.е. мб я зря думаю запихать побольше в туториал, который должен больше концепции объяснять чем этапы затаскивания в проект и голову 🤔

Copy link
Member

Choose a reason for hiding this comment

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

@unordinarity шо думаешь

Copy link
Member Author

Choose a reason for hiding this comment

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

Я тут скорее мнения, что хочется объяснить основные принципы ФСД в туториале, т.е. что такое бизнес-домен, что такое value для пользователя, как композировать компоненты. Для этого нам нужно рассказать про entities и features. Надеюсь таким образом отсечь вопросы в духе "а нотификейшн попап это фича?".

Однако размышление про то, что на маленьких проектах нахрен не нужны фичи, вполне здравое. Собственно, аппка в туториале такая простая, что она как раз показывает, насколько там не нужны фичи (или даже сущности, по факту). Поэтому я за то, чтоб после этого фундаментального туториала пустить гайд о том, как сделать нормально, без усложнений. Подумаю над подводкой к нему

Copy link
Member

Choose a reason for hiding this comment

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

Супер, спасибо за дельные размышления по теме 🚀

{
url: "https://samokat.tech/",
src: "samokat.svg",
alt: "Samokat.tech",
Copy link
Member

Choose a reason for hiding this comment

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

thoughts: current бы подлить, т.к. лого самоката уже затащили

version: VERSIONS.V2,
updatedAt: "2023-04-24",
tech: ["react", "redux-toolkit", "typescript", "vite"],
},
Copy link
Member

Choose a reason for hiding this comment

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

thoughts: И это в current уже тоже должно быть

@azinit azinit changed the title Begin writing the new tutorial FSDCUR: Begin writing the new tutorial May 10, 2023
@illright illright changed the base branch from current to master October 12, 2023 21:26
@illright
Copy link
Member Author

This PR is problematic to get up to date now, but I don't want to force push so that we don't lose comment history. I'll create a new one for this tutorial

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.

3 participants