Skip to content

Fix missing logo192.png reference. Closes #627#655

Merged
regulartim merged 1 commit intointelowlproject:developfrom
Eshaan-byte:fix-missing-logo192-develop
Jan 1, 2026
Merged

Fix missing logo192.png reference. Closes #627#655
regulartim merged 1 commit intointelowlproject:developfrom
Eshaan-byte:fix-missing-logo192-develop

Conversation

@Eshaan-byte
Copy link
Copy Markdown
Contributor

The frontend's index.html file referenced a non-existent logo192.png file for the Apple Touch Icon. This PR updates the reference to use the existing favicon.ico file instead, eliminating the 404 error that occurs when browsers attempt to load the missing asset.

Related issues

Fixes #627

Type of change

  • Bug fix (non-breaking change which fixes an issue).

Checklist

  • I have read and understood the rules about how to Contribute to this project.
  • The pull request is for the branch develop.
  • I have added documentation of the new features. (N/A - bug fix only)
  • Linters (Black, Flake, Isort) gave 0 errors. (N/A - HTML only change, no Python/JS code modified)
  • I have added tests for the feature/bug I solved. All the tests (new and old ones) gave 0 errors. (N/A - simple HTML reference fix, no functional code changed)
  • If changes were made to an existing model/serializer/view, the docs were updated and regenerated. (N/A - HTML only)

@regulartim
Copy link
Copy Markdown
Collaborator

Thanks for your contribution! :) The name of the missing file suggests that an image of size 192x192 is required. The favicon is 512x512. Do you think that's a problem?

Created a 192x192 pixel logo image from the existing greedybear.png
to serve as the Apple Touch Icon. This resolves the 404 error when
the frontend tries to load the missing asset.

Fixes intelowlproject#627
@Eshaan-byte Eshaan-byte force-pushed the fix-missing-logo192-develop branch from 4793b99 to 5dd3227 Compare January 1, 2026 14:02
@Eshaan-byte
Copy link
Copy Markdown
Contributor Author

You're absolutely right! @regulartim I've updated the PR to add the missing logo192.png file (192x192 pixels) instead of using the 512x512 favicon.

@regulartim
Copy link
Copy Markdown
Collaborator

Nice!

@regulartim regulartim merged commit d35ef32 into intelowlproject:develop Jan 1, 2026
6 checks passed
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.

2 participants