WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@ashermatsh
Copy link

Change the sound URL from "../flappy-svg/Sounds/Flappy.mp3" to "../Sounds/Flappy.mp3" and "../flappy-svg/Sounds/GameOver.mp3" to "../Sounds/GameOver.mp3"

@abishekvashok
Copy link
Member

Can I see some testing please?

@ashermatsh
Copy link
Author

Ok sure
how should I show the testing??

@abishekvashok
Copy link
Member

Show a screencast of it working? Providing a deployment link for anyone to go and test? There are plenty of ways :)

@ashermatsh
Copy link
Author

@abishekvashok
Copy link
Member

Yep seems fine. I will checkout the code locally and get back.

@ashermatsh
Copy link
Author

OK Sure✌

Copy link
Member

@abishekvashok abishekvashok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ashermatsh
Copy link
Author

ashermatsh commented Sep 14, 2021

Will it be merged?? @abishekvashok

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.

2 participants