-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Favicon defaulting to dub.co favicon in Slack #709
Comments
@nkkko Wouldn't this be considered a slack issue with slack grabbing icons off the domain directly instead of following through the redirect to the end route? |
@versecafe yeah that was my first impression as well! we could potentially polyfill this somehow, but would need to dig deeper into it |
@steven-tey I don't know if NextJS has a great way to swap the favicon programmatically BUT using a middleware layer with |
When sharing short links in Slack, favicon defaults to dub.co favicon like in the screenshot:
Slack doesn't respect the page's icon head tags and takes the
/favicon.ico
of the root domain.Defaulting back to the workspace's logo could be a solution.
The text was updated successfully, but these errors were encountered: