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

Remove complete circuit requirement from Air Powered Vertical Coaster #11449

Merged
merged 6 commits into from
Apr 26, 2020

Conversation

SpartanFrederic104
Copy link
Contributor

This updated Ride.cpp file changes the Air Powered Vertical Coaster so that it can be tested and opened without the need for a complete circuit (mirroring RCT1 and RCT3 behavior). The original RCT2 requirement for this coaster type that it be a complete circuit is merely restrictive in nature and was not present in RCT1.

… (for RCT1 parity)

This updated Ride.cpp file changes the Air Powered Vertical Coaster so that it can be tested and opened without the need for a complete circuit (mirroring RCT1 and RCT3 behavior). The original RCT2 requirement for this coaster type that it be a complete circuit is merely restrictive in nature.
@janisozaur
Copy link
Member

Please make sure your commit messages adhere to https://github.com/OpenRCT2/OpenRCT2/wiki/Commit-Messages

@janisozaur janisozaur added the network version Network version needs updating - double check before merging! label Apr 25, 2020
@duncanspumpkin
Copy link
Contributor

duncanspumpkin commented Apr 25, 2020

@Gymnasiast what do you think about making this into a flag that is part of the RTD?
.:Edit:. oh wait never mind we are removing the requirement its now just a ride->mode which is not RTD

@Gymnasiast
Copy link
Member

tupaschoal and others added 2 commits April 25, 2020 18:50
Co-Authored-By: Michael Steenbeek <m.o.steenbeek@gmail.com>
@tupaschoal
Copy link
Member

I've fixed the format and changelog, please squash on merge and mind the commit message

@tupaschoal
Copy link
Member

Ah, it needs updating network mode, according to Janis, so it's still a no go

@Gymnasiast Gymnasiast merged commit a3e7f09 into OpenRCT2:develop Apr 26, 2020
@tupaschoal tupaschoal added this to the v0.3.0 milestone Jun 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
network version Network version needs updating - double check before merging!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants