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

added keyboard controls to catch the flow #5

Closed
wants to merge 1 commit into from
Closed

added keyboard controls to catch the flow #5

wants to merge 1 commit into from

Conversation

Deepaksharm
Copy link
Contributor

any suggestion would be appretiated...

@Vigneshsekar
Copy link
Member

Hi,
You work is greatly appreciated. But, Please add the keyboard controls within the js/catchtheflow.js file and remove the folder in the name of "catchtheflow".

Since the web app is directly deployed from github, having multiple copies of same images will take us beyond the permitted size-limit for deployment in Azure.

@Deepaksharm
Copy link
Contributor Author

Thanks i will soon update the catchtheflow.js file..

On Sun, Feb 7, 2016 at 7:47 PM, Vignesh S notifications@github.com wrote:

Hi,
You work is greatly appreciated. But, Please add the keyboard controls
within the js/catchtheflow.js file and remove the folder in the name of
"catchtheflow".

Since the web app is directly deployed from github, having multiple copies
of same images will take us beyond the permitted size-limit for deployment
in Azure.


Reply to this email directly or view it on GitHub
#5 (comment).

Vigneshsekar pushed a commit that referenced this pull request Feb 16, 2016
@deepakagg
Copy link
Member

This issue was moved to #5

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