Remove WoC console.log

[pivilartisant]

Delivery Quality Checklist

• Breaking Changes in API:
  Does this PR introduce breaking changes in the API?

  • If yes, have you considered making it backward compatible?
  • If backward compatibility is not considered, set the "breaking-change" label.

• Changelog:

  • For bugfix PR, set the "bugfix" label
  • If this change should not appear in changelog, use "ignore-for-changelog" label

• Version Update Handling:
  Have you ensured that the version update by user is handled correctly?

• PR Dependency:
  Does this PR depend on another PR?

  • If yes, is it necessary for the dependency to be released prior to merging this one?

• Documentation:

  • Are any necessary changes made to user-facing documentation?
  • Confirm that API documentation is updated with any relevant changes.
  • Check that README and other documentation files are accurate and current.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (33957a0) 25.27% compared to head (eb742e6) 20.67%.

❗ Current head eb742e6 differs from pull request most recent head a90012a. Consider uploading reports for the commit a90012a to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1257      +/-   ##
==========================================
- Coverage   25.27%   20.67%   -4.60%     
==========================================
  Files          25       19       -6     
  Lines        1559     1248     -311     
==========================================
- Hits          394      258     -136     
+ Misses       1152      978     -174     
+ Partials       13       12       -1     

see 7 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[thomas-senechal]

Should we also remove those logs ?

station/web/massastation/src/pages/Search/DomainModelItem/DomainModelItem.tsx

Line 25 in 33957a0

console.log(website.name, response.status);

And this should be a console.error ?

station/web/massastation/src/pages/Search/DomainModelItem/DomainModelItem.tsx

Line 38 in 33957a0

console.log(`error fetching favicon for ${website.name}.massa`, error);

[Thykof]

Should we also remove those logs ?

station/web/massastation/src/pages/Search/DomainModelItem/DomainModelItem.tsx

Line 25 in 33957a0

console.log(website.name, response.status);

And this should be a console.error ?

station/web/massastation/src/pages/Search/DomainModelItem/DomainModelItem.tsx

Line 38 in 33957a0

console.log(`error fetching favicon for ${website.name}.massa`, error);

The first one, yes.

The second on, no, and it should be console.error