SaturnTeam / saturn-datepicker

Angular Material Datepicker with range selection
MIT License
276 stars 110 forks source link

fixes build for angular material 10.2 #156

Closed zualexander closed 3 years ago

zualexander commented 4 years ago

with angular 10.2. the build breaks:

`ERROR in node_modules/saturn-datepicker/datepicker/datepicker.d.ts:57:22 - error TS2420: Class 'SatDatepicker' incorrectly implements interface 'CanColor'. Property 'defaultColor' is missing in type 'SatDatepicker' but required in type 'CanColor'.

57 export declare class SatDatepicker implements OnDestroy, CanColor {


  node_modules/@angular/material/core/common-behaviors/color.d.ts:15:5
    15     defaultColor: ThemePalette | undefined;
'defaultColor' is declared here.`
zualexander commented 4 years ago

pls tell me if I did something wrong, or I should adapt the PR :)

itstueben commented 4 years ago

closes #157

zualexander commented 4 years ago

do you want me to change the 'defaultColor' to an '@Input'?

raphael22 commented 4 years ago

@SaturnTeam Could you have a look ? Thanks !

itstueben commented 4 years ago

I switched to default mat-datepicker. https://material.angular.io/components/datepicker/overview#date-range-input-forms-integration

georgms commented 3 years ago

This awesome date picker is currently unusable with Angular Material 10. It would be so great if you could merge this simple change so we can upgrade our apps to Angular Material 10.

gualtierim commented 3 years ago

Hi, I tried to install the new version with npm (8.0.6), but I didn't found it. Can you release the new version? Thanks to great work done!

zualexander commented 3 years ago

I'm sorry, I'm not a contributor of this repo, I just added this quickfix as a PR. @SaturnTeam can you do the release? :)

georgms commented 3 years ago

@SaturnTeam Thanks for merging the fix. Any ETA for a new NPM release including this? Really looking forward to it so I can upgrade our app to Angular 10.

davidoween commented 3 years ago

I use Angular 10, and i have the same problem. any solution?

georgms commented 3 years ago

We switched back to the builtin Angular Material date picker which, starting with version 10, also supports date ranges which was our initial reason to use the Saturn Date picker.

davidoween notifications@github.com schrieb am Di., 23. Feb. 2021, 09:57:

I use Angular 10, and i have the same problem. any solution?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/SaturnTeam/saturn-datepicker/pull/156#issuecomment-784015799, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVZVIWUWH3RCOD5DYI6QJ3TANUW7ANCNFSM4Q6DNQ3A .

davidoween commented 3 years ago

We switched back to the builtin Angular Material date picker which, starting with version 10, also supports date ranges which was our initial reason to use the Saturn Date picker. davidoween notifications@github.com schrieb am Di., 23. Feb. 2021, 09:57: I use Angular 10, and i have the same problem. any solution? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#156 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVZVIWUWH3RCOD5DYI6QJ3TANUW7ANCNFSM4Q6DNQ3A .

I need a inline datepicker .

there is no solution?

nathan-konv commented 3 years ago

Not sure and will verify tomorrow. But I got the feeling there are 2 published packages: saturn-datepicker and saturn-datepicker-ext. The latter one seems to have the fix but not the former one which I actually use. If anybody has a chance to try before I do. I am off for today :)

nathan-konv commented 3 years ago

And it seems that @angular/material v11.2.2 now offers to pick a range: https://material.angular.io/components/datepicker/overview#date-range-selection

nathan-konv commented 3 years ago

I confirm. I used the npm package saturn-datepicker-ext instead of saturn-datepicker and everything is working fine on my end. Will replace that by the newly supported range picker from angular material v11.2.2 soon.