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

fix: fonts warning #1318

Merged
merged 3 commits into from
Dec 10, 2023
Merged

fix: fonts warning #1318

merged 3 commits into from
Dec 10, 2023

Conversation

eburnette
Copy link
Contributor

Summary

I was getting this warning while running the frontend:

files in the public directory are served at the root path.
Instead of /public/fonts/soehne-buch.woff2, use /fonts/soehne-buch.woff2.

I fixed it by changing the resolver in vite.config.ts from 'public/fonts' to just 'fonts'

Change Type

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

Testing

Manual testing to verify no error and no change in how it looks.

    The warning was:
    files in the public directory are served at the root path.
    Instead of /public/fonts/soehne-buch.woff2, use /fonts/soehne-buch.woff2.
@danny-avila
Copy link
Owner

thank you! Trying this out

@danny-avila
Copy link
Owner

This test is failing (major blocker as build fails) and is not reproducible in the first place in my environment (no fonts warnings before your changes for me).

However, it still works for me if I change public/fonts to /fonts so try this and see if you get warnings and if the test fails

I would edit your PR but no permission to do so.

@danny-avila danny-avila changed the title fix: Fix a fonts warning fix: fonts warning Dec 10, 2023
@danny-avila danny-avila merged commit d4c846b into danny-avila:main Dec 10, 2023
1 check passed
shortpoet pushed a commit to shortpoet/LibreChat that referenced this pull request Dec 30, 2023
* fix: Fix a fonts warning
    The warning was:
    files in the public directory are served at the root path.
    Instead of /public/fonts/soehne-buch.woff2, use /fonts/soehne-buch.woff2.

* See if it likes /fonts better
cnkang pushed a commit to cnkang/LibreChat that referenced this pull request Feb 6, 2024
* fix: Fix a fonts warning
    The warning was:
    files in the public directory are served at the root path.
    Instead of /public/fonts/soehne-buch.woff2, use /fonts/soehne-buch.woff2.

* See if it likes /fonts better
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

2 participants