Skip to content

fix: prebuild by adding full src dir#2074

Open
CrazyWolf13 wants to merge 1 commit intoLissy93:masterfrom
CrazyWolf13:master
Open

fix: prebuild by adding full src dir#2074
CrazyWolf13 wants to merge 1 commit intoLissy93:masterfrom
CrazyWolf13:master

Conversation

@CrazyWolf13
Copy link
Copy Markdown
Collaborator

@CrazyWolf13 CrazyWolf13 commented Apr 16, 2026

Category

Bugfix

Overview

Small changes to fix the built-in compile button. Users reported it as working before, but seems like it didn't :D
Now it seems to be fully working:

grafik grafik

Issue Number

Additional Info

@CrazyWolf13 CrazyWolf13 requested a review from Lissy93 as a code owner April 16, 2026 06:36
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 16, 2026

Deploy Preview for dashy-dev ready!

Name Link
🔨 Latest commit 20d2ea4
🔍 Latest deploy log https://app.netlify.com/projects/dashy-dev/deploys/69e0837eb305b600083a85dc
😎 Deploy Preview https://deploy-preview-2074--dashy-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Owner

@Lissy93 Lissy93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this would work still :/
Because rebuild endpoint needs the dev assets

There's 2 options:

  1. Wait till the newer version is merged, and I'll try and reduce the reliance on any client-side env vars, so none of the rebuild functionality will be needed
  2. Or, bundle all of the dev stuff in here too, which seems messy and will make everything much larger.

I'd go with option 1. Rebuilding is really ugly, and I want to fully remove any need for it.

@CrazyWolf13
Copy link
Copy Markdown
Collaborator Author

Installing without --production seems to include DevDependencies, but yeah ultimately it would be a kind of hacky workaround.

I guess I'll leave this up to you to decide then, so feel free to close this.
Thanks!

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.

2 participants