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

Создать страницу настроек приложения #1645

Closed
ivnglkv opened this issue Feb 8, 2024 · 3 comments · Fixed by #1699
Closed

Создать страницу настроек приложения #1645

ivnglkv opened this issue Feb 8, 2024 · 3 comments · Fixed by #1699
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@ivnglkv
Copy link

ivnglkv commented Feb 8, 2024

Disclaimer

Отчасти эта задача описывает альтернативный подход к интерфейсу настроек описанному в #799, отчасти – описывает новую опцию.

Problem statement/business needs

В данный момент невозможно подключить плагин к серверу за пределами запрограммированной логики, которая во-первых, не даёт гибкости, а во-вторых, кажется трудно расширяемой.
Как разработчик, я хочу иметь возможность работать с несколькими версиями бэкенда для экспериментов. Сейчас это возможно только с попеременным запуском этих версий, что очевидно неудобно, потому что требует ручной остановки/запуска приложения.
Как пользователь, я хочу иметь возможность развернуть сервер где-то в своей организации и настроить к нему подключение плагина у тестировщиков.

Этого можно достичь добавив нативную для Chrome страницу настроек (документация https://developer.chrome.com/docs/extensions/develop/ui/options-page) на которой сделать возможность задать адрес бэкенд сервера для подключения. Пример как это выглядит у другого плагина:

image

Definitions of Done

  • Страница настроек доступна через меню плагина в Chrome (как на скриншоте)
    image
  • На странице настроек можно ввести адрес сервера для подключения.
  • Указанный сервер должен иметь приоритет над автоматическим алгоритмом выбора сервера. Возможно даже нужно сделать так, что если пользователь указал адрес сервера вручную, то автоматический выбор сервера не будет включаться, чтобы не было неожиданного поведения, когда локальный сервер не работает, плагин подключился к облачному, а пользователь думает, что он работает с локальным.

Dependencies or related tasks

-

Comments

Есть два варианта реализации такой страницы: в отдельном табе или в небольшом всплывающем окне. Разница описана в документации: https://developer.chrome.com/docs/extensions/develop/ui/options-page#considerations. Возможно выбор между этими двумя опциями нужно обсуждать, чтобы учесть предполагаемое направление дальнейшего развития продукта.

По сути на эту страницу можно будет внедрить и опции предполагаемые в #799, #1531.
Либо как альтернатива – наоборот, вынести выбор сервиса в интерфейс настроек описанный в #799.

@ivnglkv ivnglkv added the enhancement New feature or request label Feb 8, 2024
@Iogsotot Iogsotot added this to BackLog in JDN.AI Board via automation Feb 8, 2024
@AlexeyGirin AlexeyGirin moved this from BackLog to To do in JDN.AI Board Mar 5, 2024
@AlexeyGirin AlexeyGirin added this to the Release 3.16 milestone Mar 5, 2024
@AlexeyGirin
Copy link
Contributor

Первая версия должна быть без дизайна Семёна
Но потом - сделаем с дизайном

@Iogsotot Iogsotot self-assigned this Mar 10, 2024
@Iogsotot Iogsotot moved this from To do to In progress in JDN.AI Board Mar 11, 2024
@Iogsotot Iogsotot moved this from In progress to To do in JDN.AI Board Mar 11, 2024
@Iogsotot Iogsotot moved this from To do to In progress in JDN.AI Board Mar 28, 2024
@Iogsotot
Copy link
Contributor

Iogsotot commented Apr 2, 2024

#1699
318776591-a043b871-b80f-42cd-b016-d9c0a1aefce1

3.15.19

@Iogsotot Iogsotot moved this from In progress to QA in JDN.AI Board Apr 2, 2024
@Iogsotot Iogsotot linked a pull request Apr 9, 2024 that will close this issue
@mobilisf
Copy link

Validated on JDN v 3.15.22 Back-end v 0.2.69

image

image

JDN.AI Board automation moved this from QA to Done Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: M (5)
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants