search

fooloomanzoo / datetime-picker

A minimal picker for date and time for Polymer, that can use the native input
MIT License
78 stars 18 forks source link

New update not working. #32

Closed johanbyren closed 6 years ago

johanbyren commented 6 years ago

Hello. I updated to new version and hit some problems. It stopped working and cant set the dates.

Uncaught ReferenceError: d is not defined at HTMLElement._fromDatetime (datetime-mixin.html:491) at HTMLElement._minChanged (datetime-mixin.html:529) at HTMLElement._minChanged (calendar-element.html:444) at Object.runObserverEffect [as fn] (property-effects.html:215) at runEffectsForProperty (property-effects.html:160) at runEffects (property-effects.html:126) at HTMLElement._propertiesChanged (property-effects.html:1674) at HTMLElement._flushProperties (property-accessors.html:564) at HTMLElement.ready (property-effects.html:1632) at HTMLElement.ready (element-mixin.html:688)

image

image

fooloomanzoo commented 6 years ago

Have you tried bower update -f?

johanbyren commented 6 years ago

Yes everything is updated but stil error. I don't know what causing this. Will try to figuring it out.

fooloomanzoo commented 6 years ago

If you check the version of bower_components/property-mixins/package.json and it is 2.1.1 then it should be working. This error doesn't occur on my side

johanbyren commented 6 years ago

image It is 2.1.1. So I need to keep searching.

johanbyren commented 6 years ago

Okey think I found the problem. image My min="{{minDateTimeValue.date}}" is refering to this property. image Propably setting the new Date() to late.

fooloomanzoo commented 6 years ago

You are right, there is a mistake in the mixin. But if you still set min as an object it should be working too. There was a variable wrongly named. I updated it and it should now not throw a mistake. Please try it for your use-case

johanbyren commented 6 years ago

Thank you man! This works after i downloaded the new version on propery-mixins. You are a hero! Great job!