atls-academy / gochicus.github.io

Study playground from noob to engineer
https://gochicus.atls.academy
GNU General Public License v3.0
0 stars 1 forks source link

feat(timer): countdown timer was created using instance of Date and f… #22

Closed gochicus closed 3 years ago

gochicus commented 3 years ago

…uctions calculating difference

affects: @practice/timer-in-page

ISSUES CLOSED: #20

gochicus commented 3 years ago

Так, стоп. Тут же конкретная практика по учебному проекту, сервис по доставке еды. Почему у тебя пакет с табами отдельно таймер отдельно. Чем это обусловлено? Или ты просто по инерции сделал? 😕

Каждый пакет отражает отдельную решенную задачу . Практика по урокам, цельным проектом это не является.

themetronome commented 3 years ago

Так, стоп. Тут же конкретная практика по учебному проекту, сервис по доставке еды. Почему у тебя пакет с табами отдельно таймер отдельно. Чем это обусловлено? Или ты просто по инерции сделал? confused

Каждый пакет отражает отдельную решенную задачу . Практика по урокам, цельным проектом это не является.

Хорошо, а если я тебя попрошу задеплоить весь проект по доставке, с работающими табами, таймером, модальным окном и тд. Как тогда поступить?

gochicus commented 3 years ago

Так, стоп. Тут же конкретная практика по учебному проекту, сервис по доставке еды. Почему у тебя пакет с табами отдельно таймер отдельно. Чем это обусловлено? Или ты просто по инерции сделал? confused

Каждый пакет отражает отдельную решенную задачу . Практика по урокам, цельным проектом это не является.

Хорошо, а если я тебя попрошу задеплоить весь проект по доставке, с работающими табами, таймером, модальным окном и тд. Как тогда поступить?

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

themetronome commented 3 years ago

Так, стоп. Тут же конкретная практика по учебному проекту, сервис по доставке еды. Почему у тебя пакет с табами отдельно таймер отдельно. Чем это обусловлено? Или ты просто по инерции сделал? confused

Каждый пакет отражает отдельную решенную задачу . Практика по урокам, цельным проектом это не является.

Хорошо, а если я тебя попрошу задеплоить весь проект по доставке, с работающими табами, таймером, модальным окном и тд. Как тогда поступить?

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

Должна. Вспомни хронологию того, как мы приняли решение о рефакторинге пакетов, когда все лежали в lessons. Проект взялся в разделе курса, весь функционал ты реализовываешь исходя из полученных в разделе знаний и навыков. Что значит "полноценный функционал страницы"?

gochicus commented 3 years ago

Так, стоп. Тут же конкретная практика по учебному проекту, сервис по доставке еды. Почему у тебя пакет с табами отдельно таймер отдельно. Чем это обусловлено? Или ты просто по инерции сделал? confused

Каждый пакет отражает отдельную решенную задачу . Практика по урокам, цельным проектом это не является.

Хорошо, а если я тебя попрошу задеплоить весь проект по доставке, с работающими табами, таймером, модальным окном и тд. Как тогда поступить?

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

Должна. Вспомни хронологию того, как мы приняли решение о рефакторинге пакетов, когда все лежали в lessons. Проект взялся в разделе курса, весь функционал ты реализовываешь исходя из полученных в разделе знаний и навыков. Что значит "полноценный функционал страницы"?

Так нет же, в курсе он прямо так и говорит, что вам сходу будет трудно такое реализовать и сейчас показываю как сделать вот такое. В частности, такое он говорит в уроках про табы. По ходу уроков работа в этой же верстке и идет. Можешь привести где сказано, что это реализация проекта на основе полученных знаний? Насчет полного функционала имею в виду, что обычно, когда речь идет о полноценном проекте, то там должны работать и кнопки и события и все, что нужно для реализации. У нас имеется четыре задачи по урокам, привязанным к уже готовой верстке. Как по мне так это отработка упражнений, а не выполненный проект. Насчет рефакторинга помню только решение, что делаем в отдельных пакетах lessons и practice. Это логично - разные разделы курса.

TorinAsakura commented 3 years ago

Касаемо рефактора кода пакета всё в порядке, но я по прежнему считаю что оформить практику нужно оформить в виде пакета с сервисом доставки, где ты комплексно применяешь полученные знания. Был бы рад узнать что ближе к истине у @TorinAsakura

так как код-сплиттингу и наследованию/импорту ESM вы пока не владеете - то, да, только так

Но, практика с пакетами хороша тем, что код из них можно импортировать через зависимости в другой пакет, однако это уже другие вещи и другие миры, а по сему, да - только целостное приложение и только в один пакет

Если пакет нежизнеспособен, его нельзя запустить и протестировать - тогда какой от него толк?