Skip to content
This repository was archived by the owner on Jul 18, 2022. It is now read-only.

Add svelte#99

Draft
cofob wants to merge 103 commits intomasterfrom
cofob/svelte
Draft

Add svelte#99
cofob wants to merge 103 commits intomasterfrom
cofob/svelte

Conversation

@cofob
Copy link
Contributor

@cofob cofob commented Apr 27, 2022

Closes #95

@cofob cofob marked this pull request as ready for review April 28, 2022 14:45
@cofob
Copy link
Contributor Author

cofob commented Apr 28, 2022

@PerchunPak review pls
Это база. Сюда нужно приделать модели, api, но это будет в других PR. Тут svelte готов.

@cofob cofob requested a review from PerchunPak April 28, 2022 14:46
@cofob
Copy link
Contributor Author

cofob commented Apr 28, 2022

Ещё тут под вопросом как делать toasts без toasts.html... Это решится в будущем, наверное, но может ты сейчас готов предложить решение?


## Установка

Установите `git` и `node.js` для вашей платформы.
Copy link
Contributor

Choose a reason for hiding this comment

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

Если это не заставляет иметь зависимость node.js в рантайме, зачем пользователя заставлять компилировать? Как на счет добавить пре-компилированные файлы для моментального запуска в проде? (Это будет стороннее изменение, так что не в этом PR, просто обсуждение)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Да, можно распространять скомпилированные файлы в релизах и докерах.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Resolved?

@cofob cofob closed this Apr 28, 2022
@cofob cofob deleted the cofob/svelte branch April 28, 2022 15:07
@cofob cofob restored the cofob/svelte branch April 28, 2022 15:08
@cofob cofob reopened this Apr 28, 2022
@cofob cofob mentioned this pull request Apr 28, 2022
driver = webdriver.Chrome(service=Service(ChromeDriverManager().install()), options=chrome_options)
driver.get("http://localhost:8000/")
driver.set_window_size(1920, 1080)
sleep(10)
Copy link
Contributor

Choose a reason for hiding this comment

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

В CI sleep уже есть, зачем дублировать?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Тут sleep нужен чтобы страница загрузилась, без него будет как тут.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

А в CI sleep чтобы сервер запустился.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Нет, он не ждёт пока страница загрузится полностью.

"""Creates essential config rows."""

def handle(self, *args, **options):
"""Handle ``initconfig`` command."""
Copy link
Contributor

Choose a reason for hiding this comment

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

Такое нужно хранить в миграциях. Зачем отдельный файл.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Миграции обслуживаются самим Django, другой формат он не поймёт. Да и там нет такого функционала чтобы значения присваивать.

Copy link
Contributor

@PerchunPak PerchunPak May 2, 2022

Choose a reason for hiding this comment

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

Миграции можно писать вручную.
https://docs.djangoproject.com/en/1.8/topics/migrations/#data-migrations

@cofob cofob added enhancement New feature or request svelte Pull requests that update Svelte code labels May 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request svelte Pull requests that update Svelte code

Projects

Status: В процессе

Development

Successfully merging this pull request may close these issues.

Svelte для frontend

2 participants