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

[MIRROR] Fixes timezone offset #2424

Merged
merged 1 commit into from
Mar 17, 2024
Merged

[MIRROR] Fixes timezone offset #2424

merged 1 commit into from
Mar 17, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1453
Original PR: tgstation/tgstation#81993

About The Pull Request

I'm gonna start off by saying I am in EST, which is important to this PR.

Ever run a local server while not living in the UK and realizing that time is just frozen for you? Basically, this is why.

We have GLOB.timezoneOffset which basically the time difference between BYOND's default timezone, UTC, and the timezone the server is being hosted in, however when you open a local host you may find that this isn't the case.
image

This should be 144000 for me (results may vary on your location) because of my timezone
image
image

514 did mess a little bit with timezones, adding a new arg in time2text
image

But it also added a new var, world.timezone, which handles basically everything we want for timezone offset stuff. By default, this is UTC, which is perfect for our case.
image

So I just swapped out our non-functional time2text(0,"hh") with world.timezone, and

image

Incredible! It works!

Time is even ticking again!

image

What an amazing feat, thanks 514!

Why It's Good For The Game

Closes tgstation/tgstation#81180

Changelog

🆑 JohnFulpWillard
fix: Time on servers hosted outside of the UK should now properly have their timezone offset.
/:cl:

* Fixes timezone offset (#81993)

## About The Pull Request

I'm gonna start off by saying I am in EST, which is important to this
PR.

Ever run a local server while not living in the UK and realizing that
time is just frozen for you? Basically, this is why.

We have ``GLOB.timezoneOffset`` which basically the time difference
between BYOND's default timezone, UTC, and the timezone the server is
being hosted in, however when you open a local host you may find that
this isn't the case.

![image](https://github.com/tgstation/tgstation/assets/53777086/bd502942-250f-4b00-a7b7-1f93a6907bb2)

This should be 144000 for me (results may vary on your location) because
of my timezone

![image](https://github.com/tgstation/tgstation/assets/53777086/955a5abc-44af-4d65-b064-29ed5b46348f)

![image](https://github.com/tgstation/tgstation/assets/53777086/632ee628-5cd0-43ce-b179-4e0d66cc24f0)

514 did mess a little bit with timezones, adding a new arg in
``time2text``

![image](https://github.com/tgstation/tgstation/assets/53777086/86966b20-1b9b-47f3-93ef-62f7b8ac92d9)

But it also added a new var, ``world.timezone``, which handles basically
everything we want for timezone offset stuff. By default, this is UTC,
which is perfect for our case.

![image](https://github.com/tgstation/tgstation/assets/53777086/893231d4-79c2-4bd4-8653-b9ad41884023)

So I just swapped out our non-functional ``time2text(0,"hh")`` with
``world.timezone``, and


![image](https://github.com/tgstation/tgstation/assets/53777086/f7193ad2-0e49-41b5-ac4d-f9c28b3b5cfd)

Incredible! It works!

Time is even ticking again!


![image](https://github.com/tgstation/tgstation/assets/53777086/5b69d066-3768-495e-bf83-885d6faeabe1)

What an amazing feat, thanks 514!

## Why It's Good For The Game

Closes tgstation/tgstation#81180

## Changelog

:cl:
fix: Time on servers hosted outside of the UK should now properly have
their timezone offset.
/:cl:

* Fixes timezone offset

---------

Co-authored-by: John Willard <[email protected]>
@ReezeBL ReezeBL merged commit 36178e1 into master Mar 17, 2024
25 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-1453 branch March 17, 2024 00:10
AnywayFarus added a commit that referenced this pull request Mar 17, 2024
Iajret pushed a commit that referenced this pull request May 10, 2024
* Chance to become crab on DNA meltdown (#82985)

## About The Pull Request

Sometimes when you get a nonlethal instability consequence you will now
become crab

## Why It's Good For The Game

You should be abel to reach teh peak of evolution when messing with your
genetics

## Changelog

:cl:
add: Chance to become crab on nonlethal DNA meltdown
/:cl:

* Chance to become crab on DNA meltdown

---------

Co-authored-by: carlarctg <[email protected]>
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.

Station time doesn't increment with timezone offset
3 participants