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

Drop briefcase when leaving active state #50

Merged
merged 1 commit into from
Apr 9, 2016
Merged

Drop briefcase when leaving active state #50

merged 1 commit into from
Apr 9, 2016

Conversation

TomyLobo
Copy link
Collaborator

@TomyLobo TomyLobo commented Apr 6, 2016

Thanks to @FaceMan-THE for reporting this bug.

@BSVino
Copy link
Owner

BSVino commented Apr 7, 2016

Maybe it should be done when exiting the player active state instead? Or as well?

@TomyLobo TomyLobo changed the title Drop briefcase when entering observer mode Drop briefcase when leaving active mode Apr 9, 2016
@TomyLobo
Copy link
Collaborator Author

TomyLobo commented Apr 9, 2016

@BSVino That's the better solution, yeah. Doing both doesn't seem necessary.

@TomyLobo TomyLobo changed the title Drop briefcase when leaving active mode Drop briefcase when leaving active state Apr 9, 2016
@TomyLobo
Copy link
Collaborator Author

TomyLobo commented Apr 9, 2016

That's a nice state system in there. Did you make it or did it come with the SDK?

@TomyLobo TomyLobo added this to the v1.2.1 milestone Apr 9, 2016
@BSVino
Copy link
Owner

BSVino commented Apr 9, 2016

Nah that was Valve. Ship it.

@TomyLobo TomyLobo merged commit f8c4799 into BSVino:develop Apr 9, 2016
@TomyLobo TomyLobo deleted the fix-ghostbox branch April 9, 2016 18:16
TomyLobo added a commit that referenced this pull request Apr 20, 2016
Drop briefcase when leaving active state
shmopaloppa pushed a commit to shmopaloppa/DoubleAction that referenced this pull request Sep 13, 2017
Drop briefcase when leaving active state
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