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

Update to Polymer 2.0 preview and fix tests (dash-35) #25

Merged
merged 7 commits into from
Apr 28, 2017

Conversation

alvarezguille
Copy link
Member

@alvarezguille alvarezguille commented Feb 27, 2017

This change is Reviewable

@rogozinds
Copy link
Contributor

:lgtm:


Reviewed 11 of 11 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

alvarezguille and others added 5 commits March 15, 2017 17:33
Added name to input field in hopes that it will give the possibility to store the key in browser memory and thus suggest it the next time you see the popup. Doesn't help if you nuke the whole browser cache, but might help if you clear local storage specifically. And shouldn't hurt.
@alvarezguille
Copy link
Member Author

Reviewed 2 of 2 files at r2, 1 of 1 files at r3, 5 of 5 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@alvarezguille alvarezguille merged commit 69daa5f into master Apr 28, 2017
@alvarezguille alvarezguille deleted the 2.0-preview branch April 28, 2017 12:36
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