Feat: use workerize to manage web worker

nitin42 / react-imgpro

📷 Image Processing Component for React

2.18k stars 83 forks source link

Feat: use workerize to manage web worker #21

Closed Sunshine168 closed 6 years ago

Sunshine168 commented 6 years ago

PING @nitin42

nitin42 commented 6 years ago

Looks good to me. Have you tested workerize usage properly ?

nitin42 commented 6 years ago

Also @Sunshine168, I might not be able to review all of your changes so if you feel like the changes you've made in a PR are good to go and passes all the tests, then just merge it. But again, thanks for all the help 😃

Sunshine168 commented 6 years ago

Yep workerize work correctly. Before I think some change break the origin structure of code . HaHa then i will continue work on this

Sunshine168 commented 6 years ago

Oh , I found it break in production , i will fix it soon .

Sunshine168 commented 6 years ago

Mistake roll-back