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

Adds Multiball Ball Save for Add A Ball #1592

Merged
merged 5 commits into from
Aug 19, 2021

Conversation

atummons
Copy link
Contributor

This adds a unique ball save feature for add a ball. This uses the same
event calls, so you can trigger your shows off the same events for
normal ball save and add a ball ball save. The add a ball ball save
will not run (or add time) if the ball save is already running. But if
the normal ball save has expired, it will start this ball save at the
add a ball configured times for shoot again, hurry up, and grace period.

This adds a unique ball save feature for add a ball.  This uses the same
event calls, so you can trigger your shows off the same events for
normal ball save and add a ball ball save.  The add a ball ball save
will not run (or add time) if the ball save is already running.  But if
the normal ball save has expired, it will start this ball save at the
add a ball configured times for shoot again, hurry up, and grace period.
mpf/devices/multiball.py Outdated Show resolved Hide resolved
mpf/devices/multiball.py Outdated Show resolved Hide resolved
@jabdoa2
Copy link
Collaborator

jabdoa2 commented Aug 10, 2021

Quite a nice addition. I love it! Left some small nits.

Adds a method to handle the callbacks for ball save end, hurry up, and
grace period. This is used by normal ball save and add a ball ball save.
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jabdoa2 jabdoa2 merged commit 7d1de76 into missionpinball:dev Aug 19, 2021
@atummons atummons deleted the MultiballAddBallBallSave branch August 28, 2021 23:10
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