-
Notifications
You must be signed in to change notification settings - Fork 10
3.1.1 #22
base: master
Are you sure you want to change the base?
3.1.1 #22
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your PR!
I guess this PR should be called "migrate to Polymer 3", rather than '3.1.1', as then the version would end up being something like '3.0.0'.
I added a couple of comments from a quick reading, can you have a look?
|
||
|
||
|
||
/* Validates that the confirm password matches the entered password */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This whitespace here looks odd and excessive.
|
||
import '../@polymer/iron-validator-behavior/iron-validator-behavior.js'; | ||
|
||
const template = html` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be unused.
"input", | ||
"password" | ||
], | ||
"_requested": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This _requested
block looks wrong, accidentally left-over from some automatic tool?
is: 'paper-password-input', | ||
_template:template, | ||
properties: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indentation is off here.
Added changes to migrate it to polymer 3 application