-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
build(ui): migrate from CRA/Jest to Vite/Vitest #3311
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
Conversation
Download the artifacts for this pull request: |
@certuna with this change, I replaced our custom |
The PWA appears to work, but the name is a bit different. With previous Navidrome on iOS, I have a blue icon with N. With the new one, it's black and U. Otherwise, it seems to work fine (in neither case do I see an icon for PWA) |
Thanks for the feedback. Yeah, I need to configure the PWA manifest. Will try to do it tomorrow. |
Is the manifest supposed to look like {"0":".","1":"/","2":"p","3":"u","4":"b","5":"l","6":"i","7":"c","8":"/","9":"m","10":"a","11":"n","12":"i","13":"f","14":"e","15":"s","16":"t","17":".","18":"w","19":"e","20":"b","21":"m","22":"a","23":"n","24":"i","25":"f","26":"e","27":"s","28":"t","name":"ui","short_name":"ui","start_url":"./","display":"standalone","background_color":"#ffffff","lang":"en","scope":"./"} ? |
Where are you getting this from? |
/app/manifest.webmanifest (the manifest that Navidrome uses) |
3530fbb
to
8756b81
Compare
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
I was hoping to avoid doing this before the new UI, but everytime I have to build the UI with CRA, or start
make dev
, i die a little.