Skip to content

PRO-UI Fix default theme #835

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

PRO-UI Fix default theme #835

wants to merge 4 commits into from
+11 −16

Conversation

@endryhold
Copy link
Contributor

@endryhold endryhold commented Nov 26, 2025

Pull Request description

Changelog

  • change default theme to 2gis
  • change default maoCenter to Moscow

Issues

  • ONPREM-2729

Breaking changes

  • смена параметров по-умолчанию. При использовании параметров по-умолчанию необходимо будет явно указать требуемые.

Check-list. Чек-лист код-ревью

  • Запрос на слияние в develop.
  • Есть описание к PR.
  • Указаны блокирующие изменения. Breaking-Changes
  • Соответствие кода принятому стилю
    • Описание настроек.
    • Именование настроек.
    • Дефолтные значения.
    • Стиль кода.
  • Работоспособность. Разворачивается на своем окружении из ветки PR.
    • Тест API через тесты helmfile-хуков или коллекций Postman.
  • Не осталось мусора от удаления каких-то параметров. Ищется поиском по проекту из ветки PR.
  • Отработка линтера на чарт из ветки PR. Пример: helm lint charts/search-api

Fix default theme
b56f9c8 
- change default theme to 2gis
- change default maoCenter to Moscow
@endryhold endryhold requested review from a team as code owners November 26, 2025 07:36
r9odt
r9odt approved these changes Nov 26, 2025
View reviewed changes
Copy link
Collaborator

@r9odt r9odt left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Map watermark and menu logo changed from urbi to 2gis in UI.
Map center changed to Moscow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@r9odt r9odt r9odt approved these changes

@v-a-v v-a-v v-a-v approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@endryhold @r9odt @v-a-v