Momentjs with Typescript : "type string is not assignable to type 'Date'"

Question

Till now I was using momentjs library like this:

import * as moment from 'moment'; // import moment.
@Component({..});
export class TestClass {
  lastUpdated = Date
  constructor(private myService: MyService){
    this.lastUpdated = this.getCurrentTime(); **// this shows error.. that type string is not assignable to type 'Date'**
  }
 var getCurrentTime(){
    return moment().format('DD MMM YYYY HH:mm:ss'); 
 }
}

Above code shows me this error: that type string is not assignable to type 'Date'**

But same code exactly working fine when I use this line in place of the import module, evenn other resturent os

declare var moment: any;

But I canot use above statement as it is using "var": and 'any' is not suggested by Lint.

Why getCurrentTime is returning String? Should not this return Date

getCurrentTime() returns a string b/c moment's format() method returns a string. momentjs.com/docs/#/displaying/format — Sunil D.
– Sunil D., Commented May 17, 2017 at 20:31
The whole purpose of format is to return formatted string. What would 'formatted Date' look like? — Estus Flask
– Estus Flask, Commented May 17, 2017 at 21:08

basarat · Accepted Answer · 2017-05-18 00:37:30Z

9

“type string is not assignable to type 'Date'”

lastUpdated = Date should be lastUpdated: string.

Code

Fixed a few other things as well:

import * as moment from 'moment'; // import moment.
@Component({..});
export class TestClass {
  lastUpdated: string;
  constructor(private myService: MyService){
    this.lastUpdated = this.getCurrentTime();
  }
 getCurrentTime(){
    return moment().format('DD MMM YYYY HH:mm:ss'); 
 }
}

answered May 18, 2017 at 0:37

basarat

278k60 gold badges475 silver badges528 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Dr NVS · Accepted Answer · 2020-03-25 04:28:33Z

2

Can avoid calling one more function by the below code


import * as moment from 'moment'; // import moment.
@Component({..});
export class TestClass {
  lastUpdated: string;
  constructor(private myService: MyService){
    this.lastUpdated = moment(this.lastUpdated).format('DD MMM YYYY HH:mm:ss');
  }
}

answered Mar 25, 2020 at 4:28

Dr NVS

2314 silver badges6 bronze badges

1 Comment

Eneko Over a year ago

I think moment(this.lastUpdated) should be moment().

Collectives™ on Stack Overflow

Momentjs with Typescript : "type string is not assignable to type 'Date'"

2 Answers 2

Code

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Code

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related