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

Add the 'type' attribute on buttons to prevent auto-submission if the… #9

Merged
merged 1 commit into from Jan 3, 2017

Conversation

noreiller
Copy link
Contributor

… calendar is contained in a form

@coveralls
Copy link

coveralls commented Jan 3, 2017

Coverage Status

Coverage remained the same at 96.682% when pulling 1ff63f7 on noreiller:master into d0469ce on tomkp:master.

@tomkp tomkp merged commit c152bd1 into tomkp:master Jan 3, 2017
@tomkp
Copy link
Owner

tomkp commented Jan 3, 2017

Thanks!

@noreiller
Copy link
Contributor Author

Hi, you're welcome.
By the way, I saw you don't use tags or release, is there any reason for that ?

@tomkp
Copy link
Owner

tomkp commented Jan 9, 2017

I'm trying out tags and releases now... :-)

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.

None yet

3 participants