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

how ? #294

Closed
wants to merge 1 commit into from
Closed

how ? #294

wants to merge 1 commit into from

Conversation

Yoghurt4C
Copy link

verified by github web editor

ylvealtal ubbers
@GregoriusT
Copy link
Collaborator

GregoriusT commented Jul 24, 2023

Uhm what exactly are you doing there? You did not even bother writing anything about the Pull Request you posted, this looks like you are just trying to farm repositories for street cred or something stupid like that.

@Roadhog360
Copy link
Owner

Roadhog360 commented Jul 24, 2023

The code looks a little odd but is written the way it is for a reason. The variables are checked that way so the sounds do not repeatedly play, and only play once at a time, when the variable is flipped. I tested your commit and it causes a bug with repeatedly playing the deactivate sound when I create a beacon, which is the exact result I expected when reading your code...

Did you even test this?

I would like to ask what you're even trying to do but ylvealtal ubbers does not appear to contain any useful information...

Perhaps my code could be better, I admit. But please be sure to test your code before creating a PR :)

@Roadhog360 Roadhog360 closed this Jul 24, 2023
@Yoghurt4C
Copy link
Author

at last, I truly see

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.

3 participants