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

Fix Round Time Offset #6384

Merged
merged 1 commit into from
Jun 7, 2024
Merged

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Jun 6, 2024

About the pull request

As of byond 515.1637 time2text's behavior has changed again (see https://www.byond.com/forum/post/2922387 ). This fixes the timezone offset used for round time by ensuring time2text is parsing a positive number since it now treats 0 like it does negative numbers somreason.

Explain why it's good for the game

Fixes #6381

Testing Photographs and Procedure

Screenshots & Videos

image

Changelog

🆑 Drathek
fix: Fixed round time having the wrong timezone offset because of an engine change to time2text
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Jun 6, 2024
@Drulikar Drulikar marked this pull request as ready for review June 6, 2024 00:56
@Drulikar Drulikar requested a review from fira as a code owner June 6, 2024 00:56
@Drulikar Drulikar added the Testmerge Candidate we'll test this while you're asleep and the server has 10 players label Jun 6, 2024
cm13-github added a commit that referenced this pull request Jun 6, 2024
cm13-github added a commit that referenced this pull request Jun 6, 2024
cm13-github added a commit that referenced this pull request Jun 6, 2024
cm13-github added a commit that referenced this pull request Jun 6, 2024
cm13-github added a commit that referenced this pull request Jun 6, 2024
cm13-github added a commit that referenced this pull request Jun 6, 2024
cm13-github added a commit that referenced this pull request Jun 6, 2024
cm13-github added a commit that referenced this pull request Jun 7, 2024
@harryob harryob added this pull request to the merge queue Jun 7, 2024
Merged via the queue into cmss13-devs:master with commit 35dcb6a Jun 7, 2024
31 checks passed
cm13-github added a commit that referenced this pull request Jun 7, 2024
@Drulikar Drulikar deleted the Fix_Round_Time branch June 7, 2024 11:14
Git-Nivrak pushed a commit to Git-Nivrak/cmss13 that referenced this pull request Jun 9, 2024
# About the pull request

As of byond 515.1637 time2text's behavior has changed again (see
https://www.byond.com/forum/post/2922387 ). This fixes the timezone
offset used for round time by ensuring time2text is parsing a positive
number since it now treats 0 like it does negative numbers somreason.

# Explain why it's good for the game

Fixes cmss13-devs#6381 

# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>


![image](https://github.com/cmss13-devs/cmss13/assets/76988376/5ce5aa6f-f66f-424a-8039-b327d45909a1)

</details>


# Changelog
:cl: Drathek
fix: Fixed round time having the wrong timezone offset because of an
engine change to time2text
/:cl:
Git-Nivrak pushed a commit to Git-Nivrak/cmss13 that referenced this pull request Jun 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more Testmerge Candidate we'll test this while you're asleep and the server has 10 players
Projects
None yet
Development

Successfully merging this pull request may close these issues.

round time doesn't display how long the round has gone on for anymore
2 participants