search

runem / lit-analyzer

Monorepository for tools that analyze lit-html templates
MIT License
318 stars 36 forks source link

Complex attribute converter still triggers incompatible-type warning #260

Closed e111077 closed 2 years ago

e111077 commented 2 years ago

When setting a complex attribute converter on a property to accept complex data types e.g.:

@property({converter: {toAttribute: ..., fromAttribute: ...}, reflect: true})
date = new Date();

lit analyzer gives the following issue:

The built in converter doesn't handle the property type 'Date'. Please add '{attribute: false}' on @property decorator for 'date'lit-plugin(no-incompatible-property-type)(2303)
bicknellr commented 2 years ago

Could you expand on the example? I've been having trouble reproducing this.

e111077 commented 2 years ago

I cannot reproduce anymore. Must've been fixed somewhere along the line!