Skip to content

Conversation

@EphDoering
Copy link
Contributor

This changes the displayed date-time format on the save page from ISO UTC to the user's locale and timezone as defined by their web browser. This should fix/close issue #299.

Gweneph added 2 commits February 24, 2022 15:26
@EphDoering EphDoering mentioned this pull request Feb 24, 2022
@EphDoering
Copy link
Contributor Author

EphDoering commented Feb 25, 2022

Looks like it's trying to use version* 64 of node-sass which is no longer available prebuilt. So then it's trying to build it but is either missing the required configuration or toolchain to do so properly.

On my windows system it built without errors but it fetched node-sass version 83 rather than 64. I'm not super familiar with node module versioning or package locking so I'm not sure why mine built with 83 and the automated test tried to use 64. But in either case I don't think this is an issue with my change, and I do think this same issue is causing PR #313 to also fail even though that change only was affecting the go server.

  • edit: I no longer think this is a version number but maybe related to the instruction set?
@mroote
Copy link
Member

mroote commented Mar 14, 2022

Thanks @FrederickDoering!

@mroote mroote merged commit 188a3e4 into OpenFactorioServerManager:develop Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants