VueJS function with then

Question

I have the following method:

    getData(id){
          this.$store.dispatch('getData', {
            employeeId: this.employeeId
          }).then(() => {
            if (this.employeeData.length) {
              // here some code...
            }
          });
        },
 selectEmployee(d) {

      this.getData(d.employeeId);

      // I need to execute this only after getData has been processed...
      this.$store.dispatch('fetchHistory');
    },

So what I would like to do is something like this:

selectEmployee(d) {

          this.getData(d.employeeId).then(() => {
             // check data here and then execute the fetch
             this.$store.dispatch('fetchHistory');
          });

        },

But I get error in the then secuence, seems is not allowed there.

Any clue?

you aren't returning anything from getData

Daniel A. White
– Daniel A. White

2018-07-24 19:23:30 +00:00
Commented Jul 24, 2018 at 19:23 — Daniel A. White
– Daniel A. White, Commented Jul 24, 2018 at 19:23

akuiper · Accepted Answer · 2018-07-24 19:24:37Z

3

Return the promise in getData, then you can do getData().then():

getData(id){
  return this.$store.dispatch('getData', {
            employeeId: this.employeeId
         }).then(() => {
           if (this.employeeData.length) {
              // here some code...
           }
         });
}

answered Jul 24, 2018 at 19:24

akuiper

216k33 gold badges362 silver badges379 bronze badges

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

1 Comment

VAAA Over a year ago

Amazing!! Thanks alot

Alan Smith · Accepted Answer · 2018-07-24 19:34:09Z

2

Instead of chaining Promises, you could also use ES6s async await, which makes for cleaner more readable code.

async getData(id) {
    await this.$store.dispatch('getData', {
        employeeId: id,
    });

    if(this.employeeDate) //do something
},

async selectEmployee(d) {
    await this.getData(d.employeeId);

    // I need to execute this only after getData has been processed...
    this.$store.dispatch('fetchHistory');
},

answered Jul 24, 2018 at 19:34

Alan Smith

211 bronze badge

Collectives™ on Stack Overflow

VueJS function with then

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related