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

refactor: move title screens to assets, move Fix Lobby Screen verb to Special Verbs category #947

Merged
merged 4 commits into from
Jan 31, 2024

Conversation

Gaxeer
Copy link
Collaborator

@Gaxeer Gaxeer commented Jan 27, 2024

Что этот PR делает

Рефактор способа доставки лобби картинок пользователю с использованием datum/asset.

Почему это хорошо для игры

Должно исправить проблему, когда картинку в лобби пытается отобразить до того, как она будет получена пользователем.

Тестирование

Протестировали с другим игрком, картинка отобразилась нормально. Следующий этап тестирования уже на лайв серверах.

Changelog

🆑
tweak: Fix Lobby Screen в стат панельке был перенесен в категорию Special Verbs
fix: Картинка лобби скрина теперь должна подгружаться без проблем и черный экран вместо неё не должен отображаться
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Jan 27, 2024
@ss220bot
Copy link
Collaborator

Test Merge Deployed

Details
Server Instance

Paradise-Prime

Revision

Origin: ef0b62d
Pull Request: 68ce905
Server: 043ee503f1d243834fe29014f3457872a0a9e60b

Comment

@ss220bot
Copy link
Collaborator

Test Merge Removed

@Legendaxe Legendaxe added the Do Not Merge Не заливать. В ожидании хэда/хоста. Или просто временный тестмерж. label Jan 28, 2024
@ss220bot
Copy link
Collaborator

Test Merge Deployed

Details
Server Instance

Paradise-main-I

Revision

Origin: f0196ef
Pull Request: 28ed692
Server: 4c2e1cfabf73019fb478c2fe131e1c8f7d26fb0b

Comment

@ss220bot
Copy link
Collaborator

Test Merge Deployed

Details
Server Instance

Paradise-Prime

Revision

Origin: f0196ef
Pull Request: 28ed692
Server: 3ee0673c5fb76515b09f82cae6e1ea6f31bcd9b3

Comment

Copy link

@Furrior Furrior left a comment

Choose a reason for hiding this comment

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

Ну вроде норм

@Furrior Furrior merged commit a0eb341 into ss220club:master Jan 31, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Не заливать. В ожидании хэда/хоста. Или просто временный тестмерж. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants