internetarchive / wayback-diff Goto Github PK

Thanks for the commit cfe567d expanding on the provenance of this code. It is also important to add web-monitoring-ui's GPL license to this repository. You may simply copy this file.

Before showing the sunburst, we validate the target timestamp.
The way we do it currently is that we load the capture and if its successful, we conclude that the timestamp is valid.
This method is very inefficient, the capture could be really big, its not good trying to download it all.
Please perform a CDX query instead using the URL and timestamp to see if its valid.

Please also add a short comment in this function to explain its purpose.

We could change showError to error and have the following logic:
when error is None, its equal to showError=false, when there is an error, set a msg string to error and use that in <ErrorMsg> component to display custom error messages.

Its not good that <ErrorMsg> has a fixed msg for everything besides status=404.

From my understanding we implement base64 decoding in our own code,
can we use standard function atob ?
https://www.w3schools.com/jsref/met_win_atob.asp
(Browser support is OK, we don't need to support old browsers).

I guess there should be many other libraries in npm like https://www.npmjs.com/package/js-base64

Please also add a comment in the function saying why we need to base64 decoding.

enzyme is deprecated and does not support React 18.
This is blocking us from upgrading to React 18.
We need to replace it with a more modern unit testing library.
Guide: https://wanago.io/2022/04/04/enzyme-react-testing-library/

Reimplement the d3 scale cluster in a simpler way with d3 without any external dependencies.

We need to have a leaner package with less dependencies for better maintenance and efficiency.