search

PandoraPay / PandoraPay-wallet

Zero Wallet
4 stars 10 forks source link

No need for two distinct web worker bundles... #16

Closed yuansong862 closed 2 years ago

yuansong862 commented 2 years ago

After further review, we just noticed that you don't need two separate bundles for running the two web workers. This PULL https://github.com/PandoraPay/PandoraPay-wallet/pull/15 saves ~40kb and simplifies the overall architecture.

ibudisteanu commented 2 years ago

Basically your proposal in pull request #15 is to remove the second web worker file and to keep it identical with the first one ?

ibudisteanu commented 2 years ago

But for example, Pandora Storage is initialized for the 2nd web worker which doesn't need it