favicons icon indicating copy to clipboard operation
favicons copied to clipboard

7.0.0 is not working on Windows

Open rtarnaud opened this issue 1 year ago • 1 comments

...as opposed to 7.0.0-beta.4 for example, which does work on Windows.

rtarnaud avatar Sep 14 '22 15:09 rtarnaud

Any details?

andy128k avatar Sep 15 '22 10:09 andy128k

I'm testing on mac and it isn't working for me either. No errors are being thrown but my favicons are just not being generated. 7.0.0-beta.1 and 7.0.0-beta.4 also works for me. 7.0.0-beta.5 doesn't work for me either.

hahnbeelee avatar Sep 23 '22 21:09 hahnbeelee

May you provide more details? How do you test? May it happen that you use a callback to get a result? Callbacks are not supported anymore (since beta.5).

favicons(source, options, (error, result) => { ... }) should be changed to favicons(source, options).then(result => { ... }, error => { ... }).

andy128k avatar Sep 25 '22 16:09 andy128k

No I didn't update my code to reflect the switch to promises so I guess that's the issue. Thanks for the heads up but I would humbly suggest that you announce these kind of changes in a changelog document, not only in the readme. Especially when you change the API itself between a beta version and the corresponding release.

rtarnaud avatar Sep 25 '22 19:09 rtarnaud

The change also isn't reflected on the npm readme

hahnbeelee avatar Sep 25 '22 19:09 hahnbeelee