Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

🔗 Updated pigeon links in head.js #693

Merged
merged 1 commit into from
Oct 23, 2024
Merged

🔗 Updated pigeon links in head.js #693

merged 1 commit into from
Oct 23, 2024

Conversation

jeoffreyfischer
Copy link
Member

Closes #682

Updated broken links that were pointing to the old VM

@jeoffreyfischer jeoffreyfischer self-assigned this Oct 22, 2024
@jeoffreyfischer jeoffreyfischer merged commit 26a5e2b into main Oct 23, 2024
2 checks passed
@jeoffreyfischer jeoffreyfischer deleted the 682-update-pigeon-links-in-head-js branch October 23, 2024 05:51
@jeoffreyfischer
Copy link
Member Author

jeoffreyfischer commented Oct 23, 2024 •
edited
Loading

✅ The links have been updated and are now working.
However, running a favicon checker on SSW people results in some errors: https://realfavicongenerator.net/checker
A new PBI was created to fix that: #696
Note: this was already a problem before the current PR was merged.

Screenshot
image
Figure: Favicon checker outputs some errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tiagov8 tiagov8 tiagov8 approved these changes

@bradystroud bradystroud Awaiting requested review from bradystroud

Assignees

@jeoffreyfischer jeoffreyfischer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

SSW.People - Head - Update links pointing to the decommissioned SSW.Website server
2 participants
@jeoffreyfischer @tiagov8