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

Add an option to use a custom Favicon #1597

Open
kayvanaarssen opened this issue Nov 3, 2023 · 5 comments
Open

Add an option to use a custom Favicon #1597

kayvanaarssen opened this issue Nov 3, 2023 · 5 comments
Labels
enhancement self-hosted Applies to self-hosted instances

Comments

@kayvanaarssen
Copy link

Can you make it an option in the config file to use a custom favicon.
Either link to one, or link to a location inside the docker container (mount location)

@pglombardo
Copy link
Owner

Hi @kayvanaarssen - you can configure the favicons with these settings. Make sure to set the icon_57x57 to something as the customisation is keyed off of that value.

@kayvanaarssen
Copy link
Author

Thanks that did the Trick, but I did link it like this:

- type: bind
        source: './custom/favicon'
        target: '/opt/PasswordPusher/app/assets/images/favicon'

Than in the .env i've set the path like: /opt/PasswordPusher/app/assets/images/favicon/favicon.png

It did not like that...
So for now i've just linked them to our public website to grab the Favicon but is this a bug or am I doing something wrong here?

@pglombardo
Copy link
Owner

Yeah I should probably add a note about that. This is complexity with the app framework. If you place a file in assets/images, then assets need to be precompiled, fingerprinted and the asset needs to be loaded with asset_path.

We don't do that for custom icons - we just put the raw value on the page so it has to be a fully qualified HTTPS URL.

@kayvanaarssen
Copy link
Author

Right, did not know that😉 Not using the pre compile option so that's why it did not work.
Now loading it directly from our website so that's also fine.

@pglombardo
Copy link
Owner

HI @kayvanaarssen - I'll close this out soon if nothing remains. BTW I saw the other issue you filed - I'll have a closer look at it over the weekend.

@pglombardo pglombardo added the self-hosted Applies to self-hosted instances label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement self-hosted Applies to self-hosted instances
Projects
None yet
Development

No branches or pull requests

2 participants