tgreenx

Results 178 comments of tgreenx

Same, sounds good to me. I could see a heat map in such an History page, it is usually visually helpful.

Release testing v2024.2: no issues found.

It seems that this document is not used in any Zonemaster component anymore. Instead of updating it, we can probably remove it. ``` $ ack "Translation-translators" ../ ../zonemaster-engine/MANIFEST 4:docs/Translation-translators.md ```

As discussed, courtesy of a former contributor of this project, here are some points (with minimal redacting) related to possible regressions in accessibility. Also, I tried to group them by...

> Regarding the FAQ page. My guess is you're running node 18 or lower, we only support 20+ as 18 is EOL. My bad for not mentioning this anywhere. It...

> > Regarding the FAQ page. My guess is you're running node 18 or lower, we only support 20+ as 18 is EOL. My bad for not mentioning this anywhere....

FYI I've just edited my initial review to add one more thing in the "Regressions -> Result page" category: - When expanding the results of a test, the associated test...

@matsduf Can (or does) the current rewrite of DNSSEC10 (https://github.com/zonemaster/zonemaster/pull/1179) take into account this issue?

Solved by #1359 (release v2024.2)