-
Notifications
You must be signed in to change notification settings - Fork 736
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
Markers - Add the option for UTC/System time timestamps #9121
Conversation
Added UTC Time zone Added UTC time offset slider Added translation in stringtable
Fixed mistake that added offset twice
- Fixed calculating error
Was a bit enthusiastic with removing tabs. This should to the trick.
Hope this fixed the "Tab detected at ..." problem. Otherwise I have no idea how to fix it. It has maybe to do with my VSCode sqf formatting plugin. |
- Checked SQF with sqf_validator.py - Fixed decimal error by CBA menu
- This does not belong here
Added empty line back
Formatting update Co-authored-by: Grim <69561145+LinkIsGrim@users.noreply.github.com>
Formatting update Co-authored-by: Grim <69561145+LinkIsGrim@users.noreply.github.com>
Added newline at end of file
Fixed fnc header formatting
Formatting Co-authored-by: Grim <69561145+LinkIsGrim@users.noreply.github.com>
Updated formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment, but it's fine as is too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Modulo could also work, but it would need some testing - I tried to use it and it broke with big offsets.
Allow UTC minutes offset for marker timestamp
@johnb432 I'm finished. Hope this works. I've tested it and it worked great for me. Good luck 😄 |
…nto markerSysTime
…nds offset" This reverts commit d2599bd.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me - @LinkIsGrim can you take a second look?
When merged this pull request will:
Discussion: #9119
IMPORTANT
Component - Add|Fix|Improve|Change|Make|Remove {changes}
.