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

demonstrate reacting to form's submitted state changes #73

Closed
wants to merge 1 commit into from

Conversation

jab
Copy link
Contributor

@jab jab commented Oct 31, 2015

First pass at #68

@jab
Copy link
Contributor Author

jab commented Oct 31, 2015

@azakus Looks like this Travis build failed for unrelated reasons. Could you force trigger a new build for this PR once the build is fixed?

@dfreedm
Copy link
Contributor

dfreedm commented Nov 2, 2015

@jab, you need to rebase with master branch to get the travis config.

@jab
Copy link
Contributor Author

jab commented Nov 3, 2015

Thanks @azakus, the build is passing now.

@@ -21,7 +21,8 @@
"ignore": [],
"dependencies": {
"polymer": "Polymer/polymer#^1.0.0",
"iron-ajax": "PolymerElements/iron-ajax#^1.0.0"
"iron-ajax": "PolymerElements/iron-ajax#^1.0.0",
"paper-spinner": "PolymerElements/paper-spinner#~1.0.3"
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a dev dependency, since it's used in the demo and not the element itself :)

@notwaldorf
Copy link
Contributor

Closing, since this is an old PR and it has unaddressed comments. Please re-open when it's ready for another review.

@notwaldorf notwaldorf closed this Feb 8, 2016
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.

None yet

3 participants