fix: this.queue.push is not a function about react-advertising HOT 2 CLOSED

@filipecosta01

@pahund I was going through the code and I spotted that the queue is being set to an object instead of an array in the unmount function of lifecycle.
I can open a PR to fix that, though I'm not sure how to replicate it consistently.

That's a great catch, please do open a PR!

Do you have an alternative way to test?

Not yet. We could set up a Storybook story where we set up and tear down at the click of a button, then write a Cypress test. But for now, it will suffice if you change these blatantly wrong curly braces into square brackets.

from react-advertising.

@pahund I was going through the code and I spotted that the queue is being set to an object instead of an array in the unmount function of lifecycle. It could explain that during the unmount we still have some calls to activate and as the queue is now an object, we don't have the push function
.

https://github.com/eBayClassifiedsGroup/react-advertising/blob/3b62c84dc9c192dd7d8c5fc74df237dde2701941/src/Advertising.js#L94

I can open a PR to fix that, though I'm not sure how to replicate it consistently.
Do you have an alternative way to test?

from react-advertising.