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 lock clock #2201

Merged
merged 4 commits into from
Jun 11, 2023
Merged

Fix lock clock #2201

merged 4 commits into from
Jun 11, 2023

Conversation

scooterboo
Copy link
Contributor

Description

Ignores ClientLockGameTimeNotify so that lock time works correctly. Polls PROP_IS_GAME_TIME_LOCKED at login to find out if you are in the middle of a quest with time locking. Much thanks to @LloydTheGreenNinjagc and @Hartie95

Ya, I can't make heads or tails of what ClientLockGameTimeNotify does except it always is sending a false.

Type of changes

  • Bug fix
  • New feature
  • Enhancement
  • Documentation

Checklist:

  • My code follows the style guidelines of this project
  • My pull request is unique and no other pull requests have been opened for these changes
  • I have read the Contributing note and Code of conduct
  • I am responsible for any copyright issues with my code if it occurs in the future.

Copy link
Contributor Author

@scooterboo scooterboo left a comment

Choose a reason for hiding this comment

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

Yep

@scooterboo scooterboo requested a review from KingRainbow44 June 9, 2023 20:25
Copy link
Contributor Author

@scooterboo scooterboo left a comment

Choose a reason for hiding this comment

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

test

@KingRainbow44 KingRainbow44 merged commit 15699e5 into Grasscutters:development Jun 11, 2023
@scooterboo scooterboo deleted the patch-4 branch June 16, 2023 05:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants