Skip to content
This repository has been archived by the owner on Oct 29, 2019. It is now read-only.

add wiki page about file structure #2

Open
Eoksni opened this issue Jun 5, 2017 · 23 comments
Open

add wiki page about file structure #2

Eoksni opened this issue Jun 5, 2017 · 23 comments
Assignees

Comments

@Eoksni
Copy link
Member

Eoksni commented Jun 5, 2017

No description provided.

@Eoksni Eoksni self-assigned this Jun 5, 2017
@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

@ITarako глянь может есть что дополнить

@ITarako
Copy link

ITarako commented Jun 5, 2017

directories.doc

Put markdown files in here. Eventually, these will be displayed nicely, maybe, someday.

Это все таки для документации, а не для макетов и т.п.

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017 via email

@ITarako
Copy link

ITarako commented Jun 5, 2017

видел вариант с директорией template, но это совсем будет ад х)

и надо наверное указать для какого типа проектов, эта структура

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

нууу можно и template, но помоему это уже черезчур тщательное разделение

а что значит "для какого типа проектов" ? для всех :)

@ITarako
Copy link

ITarako commented Jun 5, 2017

подумать над директорией для темлейтов

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

А собственно, что тут думать.
frontend/src/templates
м?

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

Зато я вспомнил про картинки.

Во первых, статичные картинки для сайта, вроде логотипа и т.д. Их я думаю в /frontend/src/static.

Во вторых, ресурсы, загружаемые пользователем. Напр. картинки в случае димановского инстаграмма. Их запилнуть в /assets.

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

Хотя картинки сложно считать исходниками. Наверное, лучше все таки /frontend/static.

@ITarako
Copy link

ITarako commented Jun 5, 2017

но тот же экспресс относит к статике *.html, *.css

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

его проблемы

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

а если более развернуто, то для экспресса статик - это то, что не требует обработки, что можно сервить юзеру "как-есть". в нашем случае css и html требуют обработки в общем случае на этапе компиляции.

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

в нашем случае экспрессовский статик - это папка dist

@ITarako
Copy link

ITarako commented Jun 5, 2017

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

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

Да, когда оно все разложено по полочкам это всегда лучше чем адовый хаос :) зацени я на вики замутил мега арт для отображения этой структуры.

@ITarako
Copy link

ITarako commented Jun 5, 2017

чотко, осталось как-то пацанов на это дело принудить х)

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

добавил /backend/migrations для db миграций

@ITarako
Copy link

ITarako commented Jun 5, 2017

/backend/migrations/001_init_db.js вместо domain/src/db.async.proxy.js или как

@Eoksni
Copy link
Member Author

Eoksni commented Jun 5, 2017

нет, твой db.async это модуль для работы с базой

а init_db.js это скрипт, инициализирующий базу (создающий таблицы, поля, триггеры). Для монго это тебе пока не важно, он schemeless, но для SQL баз миграции важны. Да и для монго потом тоже понадобятся.

@Eoksni
Copy link
Member Author

Eoksni commented Jun 6, 2017

а ведь при такой схеме

frontend
├───webpack.config.js
└───.eslintrc.json

получается, что webpack.config.js попадает под действие .eslintrc.json, хотя на самом деле тот для фронтенд проекта. Нехорошо. Напр., в еслинт конфиге для фронтенда нужно определять env: browser и т.д., которые для webpack.config.js не подходят.

@Eoksni
Copy link
Member Author

Eoksni commented Jun 6, 2017

Сдвинул .eslintrc.json в src

@Eoksni
Copy link
Member Author

Eoksni commented Jun 8, 2017

Добавил пункт насчет еслинтигнора в соответствии с emotz/kladovka#38

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants