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

Отображение даты в КПК #47

Merged
merged 11 commits into from
Jun 8, 2024

Conversation

Rinary1
Copy link
Contributor

@Rinary1 Rinary1 commented Jun 8, 2024

About the PR

Вы когда-то испытывали проблемы с датой в документах? Особенно если на вики не указано какая именно дата должна быть, есть решение! Дата в пда, такое решение было у корвахов, но так как корвахи долгие не проверяющие зануды, можно не ждать принятия, т.к пулл висит с января 10 и никто не принимает(анлак).

Why / Balance

Удобство, мелочь, но приятно.

Media

image

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

  • add: Added Time in Pda

Copy link
Collaborator

@VigersRay VigersRay left a comment

Choose a reason for hiding this comment

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

Круто, но думаю стоит разбить это на две строки, отдельно длительность смены и отдельно текущее время с датой, это добавить еще больше погружения.
Кстати помню в ПРах нянотразен была штука от рейна на подобное, даже с работающими часами.
Короче думаю суть ты уловил, единственное нужно решить показывать реальное время клиента (тогда у игроков будет рассинхрон), сервера, или начинать смену всегда с 12:00.
Лично я за вариант чтобы у всех было время сервера.

@VigersRay VigersRay changed the title Date in pda Отображение даты в КПК Jun 8, 2024
@Rinary1

This comment was marked as off-topic.

@VigersRay
Copy link
Collaborator

Круто, но думаю стоит разбить это на две строки, отдельно длительность смены и отдельно текущее время с датой, это добавить еще больше погружения. Кстати помню в ПРах нянотразен была штука от рейна на подобное, даже с работающими часами. Короче думаю суть ты уловил, единственное нужно решить показывать реальное время клиента (тогда у игроков будет рассинхрон), сервера, или начинать смену всегда с 12:00. Лично я за вариант чтобы у всех было время сервера.

Сервера в Кирибати?)

На данный момент там стоит таймзона МСК, и полагаю даже если мы поменяем локацию я поставлю там ту же зону (Так куда искать по логам)

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jun 8, 2024

Круто, но думаю стоит разбить это на две строки, отдельно длительность смены и отдельно текущее время с датой, это добавить еще больше погружения. Кстати помню в ПРах нянотразен была штука от рейна на подобное, даже с работающими часами. Короче думаю суть ты уловил, единственное нужно решить показывать реальное время клиента (тогда у игроков будет рассинхрон), сервера, или начинать смену всегда с 12:00. Лично я за вариант чтобы у всех было время сервера.

Окей, сделаю часы упираясь на нянотразен, но мне кажеться легче было бы уже взять у них фулл пр(спасибо, у них только часы занимают 33 файла и 600 строк, мне кажеться это того не стоит)

@Rinary1 Rinary1 marked this pull request as draft June 8, 2024 22:03
@Rinary1 Rinary1 marked this pull request as ready for review June 8, 2024 22:53
@Rinary1
Copy link
Contributor Author

Rinary1 commented Jun 8, 2024

Сделал как ты хотел, время начинается с 12:00, дата в отдельной колонке, систему вывел в отдельный файл. Скрины прикреплю к начальному сообщению через 5-10 минут

@Rinary1

This comment was marked as off-topic.

@VigersRay VigersRay merged commit 5d34b76 into space-sunrise:master Jun 8, 2024
7 of 11 checks passed
@Rinary1 Rinary1 deleted the Date-In-Pda branch June 9, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants