Skip to content

Added online hit_ball gameplay [networking] #209

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

Closed
wants to merge 6 commits into from
Closed

Added online hit_ball gameplay [networking] #209

wants to merge 6 commits into from

Conversation

osse101
Copy link
Contributor

@osse101 osse101 commented Mar 3, 2014

netswing: synchronize players at ball and balls being hit. Tells server when players go back to cart.
netTransferToSwing: tells players when to go to ball.

This patch adds ball hitting to online mode. synchronization #20 involves balls being hit, rotation in golf mode, and PlayerSwap was moved around a bit.

Features: range finder on switching to golf mode, power meter on swing, ball hitting.

and I removed the old map indicators. #187 since the new icons are now in #85 .

@osse101
Copy link
Contributor Author

osse101 commented Mar 3, 2014

I'd like to change the switch to golf mode so it faces where you were facing before, and there's a minor inconvenience where (server player only?) must press G twice the first time they want to switch modes.

osse101 added 2 commits March 3, 2014 10:22
PowerMeter chunks weren't going away while not in use
@Cheeseness
Copy link
Contributor

After testing with @osse101, we're going to leave this one a little until some kinks are ironed out. I'll leave the PR here to give it a little extra visibility.

osse101 added 3 commits March 3, 2014 15:36
Golf mode: Space to hit, E to not hit
Golf mode synchronization extrapolation
Auto-aim on flag
Corrected client side other player animations
@osse101 osse101 closed this Mar 4, 2014
@osse101 osse101 mentioned this pull request Mar 4, 2014
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.

2 participants