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

Add discord link requie #2356

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

Werzet
Copy link
Collaborator

@Werzet Werzet commented Dec 15, 2024

Описание PR

Начало итерации по введению обязательной привязки дискорда.

  • Добавлен функционал по получению ссылок привязки от бэка личного кабинета.
  • Добавлено ожидание и проверка наличия привязки перед входом.
  • Добавлено окошечко для получения ссылки.

Медиа

Проверки

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

Изменения

🆑

  • add: Добавлена новая привязка аккаунта к дискорду через личный кабинет.

@github-actions github-actions bot added Changes: UI Изменение затронуло файлы ".xaml" Changes: C# Изменение затронуло файлы ".cs" labels Dec 15, 2024
@UrPrice
Copy link
Collaborator

UrPrice commented Dec 15, 2024

@Werzet везде, где не наш неймспейс - забыл метки проставить

Content.Client/Mapping/MappingManager.cs Outdated Show resolved Hide resolved
Content.Client/Mapping/MappingState.cs Outdated Show resolved Hide resolved
Content.Server/Mapping/MappingManager.cs Outdated Show resolved Hide resolved
Content.Shared/Corvax/CCCVars/CCCVars.cs Outdated Show resolved Hide resolved
@stalengd stalengd self-assigned this Dec 15, 2024
@Werzet Werzet requested a review from stalengd December 16, 2024 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: UI Изменение затронуло файлы ".xaml" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants