Image-sequencer: Changing asynchronous callbacks of Sequencer Object to Promises

Created on 20 Jan 2020  路  4Comments  路  Source: publiclab/image-sequencer

As of now we support ES6 but we still have so many callbacks and callback hells! We should change them to Promises because promises:

  • *[x] Allow cleaner and better, functional code with less error-prone boilerplate
  • *[x] Are easy to debug
  • *[x] Provide catching mechanism which are not in callbacks
  • *[x] Promises are built over callbacks
discussion

Most helpful comment

I think you can go ahead after listening to @jywarren .

All 4 comments

@publiclab/is-reviewers @jywarren @HarshKhandeparkar
What are your views?

I think this can be a great step towards easy debugging and attaining a more managed codebase, easier to handle. What do you think @jywarren ?

Also, @blurry-x-face do you want to work on this one?

Yes! I want to work on it!

I think you can go ahead after listening to @jywarren .

Was this page helpful?
0 / 5 - 0 ratings

Related issues

kindanduseful picture kindanduseful  路  5Comments

harshkhandeparkar picture harshkhandeparkar  路  5Comments

VladimirMikulic picture VladimirMikulic  路  3Comments

harshkhandeparkar picture harshkhandeparkar  路  3Comments

waridrox picture waridrox  路  5Comments