Skip to content

Conversation

@jarofgreen
Copy link
Contributor

No description provided.

@jarofgreen jarofgreen requested a review from Bjwebb December 11, 2025 15:49
@jarofgreen jarofgreen self-assigned this Dec 11, 2025
@jarofgreen jarofgreen marked this pull request as draft December 11, 2025 15:49
@jarofgreen
Copy link
Contributor Author

jarofgreen commented Dec 11, 2025

A test is failing with a XSD error and I still haven't tested the 2 special routes with a _ in them (the url of one will need tweaking) or the downloads

@kd-ods
Copy link
Contributor

kd-ods commented Dec 16, 2025

@jarofgreen - is it possible to do this this week, or is it too gnarly?

This removes the favicon app and does not replace it with anything.
However, this is broken on the live site anyway so no features are lost.
@jarofgreen jarofgreen marked this pull request as ready for review December 16, 2025 12:09
@jarofgreen
Copy link
Contributor Author

Ready for review!

I put it on dev server.

I checked downloads work by downloading from real and laptop and diffing.

@jarofgreen jarofgreen merged commit 7c07d6d into live Dec 17, 2025
2 checks passed
@jarofgreen jarofgreen deleted the upgrade-deps branch December 17, 2025 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants