Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Support Multiple WebUI #147

Draft
wants to merge 11 commits into
base: feature/v2_carbonfay_round1
Choose a base branch
from

Conversation

olddeda
Copy link
Contributor

@olddeda olddeda commented Aug 29, 2019

@evgeny-boger
Copy link
Member

А как пользоваться .env ? Я поменял файл, ничего не поменялось. Ну и по-умолчанию надо делать выключенной эту опция, потому что по умолчанию она для контроллера собирается.

@olddeda
Copy link
Contributor Author

olddeda commented Aug 30, 2019

А как пользоваться .env ? Я поменял файл, ничего не поменялось. Ну и по-умолчанию надо делать выключенной эту опция, потому что по умолчанию она для контроллера собирается.

Поправил баг. Теперь отображает правильно.
По поводу использования, изменить в .env WEBUI_MULTIPLE false или true и перезапустить webpack

@evgeny-boger
Copy link
Member

собрал, запустил веб-сервер в dist (до этого проверял через npm run server). Что-то с локализацией:

image

@olddeda
Copy link
Contributor Author

olddeda commented Sep 1, 2019

собрал, запустил веб-сервер в dist (до этого проверял через npm run server). Что-то с локализацией:

image

Поправил webpack, так же убрал конфликты

@evgeny-boger
Copy link
Member

image
надо порт по-умолчанию ставить 18883.

Мне ещё кажется, что надо разрешить пустой name и тогда вместо него использовать hostname. Или при отображении, или только при сохранении - не принципиальо.

@evgeny-boger
Copy link
Member

Хочется отсюда
image

как-то попадать в сам список. Как это визуально сделать - я не понял, но поймал себя на мысли, что несколько подряд искал именно там, как добавить новый сервер.

@evgeny-boger
Copy link
Member

тултип при наведении "choose a control" странный

@evgeny-boger
Copy link
Member

на мобильниках разорвало менюшку, а в списке появилась прокрутка.

image

@olddeda
Copy link
Contributor Author

olddeda commented Sep 2, 2019

Хочется отсюда
image

как-то попадать в сам список. Как это визуально сделать - я не понял, но поймал себя на мысли, что несколько подряд искал именно там, как добавить новый сервер.

сделал

@olddeda
Copy link
Contributor Author

olddeda commented Sep 2, 2019

image
надо порт по-умолчанию ставить 18883.

Мне ещё кажется, что надо разрешить пустой name и тогда вместо него использовать hostname. Или при отображении, или только при сохранении - не принципиальо.

сделал

@olddeda
Copy link
Contributor Author

olddeda commented Sep 2, 2019

тултип при наведении "choose a control" странный

поправил

@evgeny-boger
Copy link
Member

когда false, то зачем-то есть name:
image

@evgeny-boger
Copy link
Member

похоже при добавлении аккаунта role (access level) берётся из текущих настроек. А дальше изменить или посмотреть сохранённый access level нельзя, изменение в соответствуей странице меняет только на одну сессию.

Думаю надо

  • Добавить явно access level в создание/редактироваие/просмотр аккаунта
  • На страницу access level написать, что это acess level для такого-то аккаунта, при сохранении обновлять аккаунт

@sikmir sikmir marked this pull request as draft July 30, 2024 06:42
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.

2 participants