Skip to content

Conversation

@Eshaan-byte
Copy link

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)

Updated the apple-touch-icon link to reference the existing favicon.ico
instead of the non-existent logo192.png file, resolving the 404 error
when the frontend tries to load the missing asset.

Fixes intelowlproject#627
@regulartim
Copy link
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?

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