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.

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

Always show favicon #474

Merged
merged 1 commit into from
Jul 30, 2023
Merged

Conversation

argosopentech
Copy link
Contributor

Currently the LibreTranslate web app only shows
the favicon icon when the LT_WEB=1 environment
variable is set. LT_WEB is used to disable
metadata by default to prevent search engines
from indexing private LibreTranslate instances.
This commit makes the favicon available even when
the metadata is disabled.

Currently the LibreTranslate web app only shows
the favicon icon when the `LT_WEB=1` environment
variable is set. LT_WEB is used to disable
metadata by default to prevent search engines
from indexing private LibreTranslate instances.
This commit makes the favicon available even when
the metadata is disabled.

- LibreTranslate#471
@pierotofy
Copy link
Member

Excellent, thanks!!

@pierotofy pierotofy merged commit a1d8a09 into LibreTranslate:main Jul 30, 2023
4 checks passed
@PJ-Finlay PJ-Finlay deleted the show-favicon branch July 30, 2023 15:26
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.

None yet

3 participants