danielmoncada / date-time-picker

Angular Date Time Picker (Responsive Design)
https://daniel-projects.firebaseapp.com/owlng/date-time-picker
MIT License
141 stars 61 forks source link

On installing the date-time-picker #86

Closed monis001 closed 3 years ago

monis001 commented 3 years ago

Hi, I have an ANgular8+ project and on installing the date-time-picker, it's generating errors.

image

PS: I believe this repo is actively maintained, and the repo from which is forked seems not maintained, or no communication from maintainers. If anyone could guide.

Thanks

jcompagner commented 3 years ago

are you sure we talk here about this date picker? it seems more that you are using aggrid .. and it complains for that.

monis001 commented 3 years ago

Hi, That's the folder name, if you will check ut closely it is pointing out in date-picker modules, please check the below image : Angular (v : 8+) date-picker-2

(Yellow box pointing the project path)

also when tried with Angular-10, still showing the error :

date-time-picker-issue

jcompagner commented 3 years ago

right i was confused because ng-pick-datetime is not the current package of this product. but this product has a devDepedency on https://github.com/danielmoncada/date-time-picker/blob/master/package.json#L57 angular/cdk

So it could be that that is just a requirement (in our product we also use that) including also moment

danielmoncada commented 3 years ago

@monis001 your first two screenshots are using the wrong package. Package should be @danielmoncada/angular-datetime-picker not ng-pick-datetime.

@jcompagner is right. In your last screenshot, looks like you're using the correct package but you're missing dev dependencies. you need to install those manually either through NPM or Yarn.

igorlino commented 3 years ago

@danielmoncada I also came to the same problem with the cdk dependency, I think it may be a good thing to document it in the README

npm install @angular/cdk --save npm install @danielmoncada/angular-datetime-picker --save

danielmoncada commented 3 years ago

@igorlino good idea. i'll add it next rev. thanks!

monis001 commented 3 years ago

Yes....

After googling came across there is one missing package, later installed that. It will be helpful if doc gets updated.

Thanks