Skip to content

Conversation

missytake
Copy link
Contributor

@missytake missytake commented Aug 19, 2025

  • Some kind of test would be nice, as the upload logic is kinda intricate. Maybe put it into a shorter function which can be tested.

fix #529

based on #617, mostly because they share a README section. But better review/merge that one before.

@missytake missytake force-pushed the upload-custom-www-529 branch 2 times, most recently from 0538605 to d8fc315 Compare August 19, 2025 10:41
@missytake missytake changed the title WIP: www: introduce www_folder config item www: introduce www_folder config item Aug 19, 2025
@missytake missytake force-pushed the upload-custom-www-529 branch from d8fc315 to 3bf441e Compare August 27, 2025 06:56
@missytake missytake changed the base branch from custom-nginx-562 to main August 27, 2025 06:56
@missytake missytake requested a review from link2xt August 27, 2025 06:58
@missytake
Copy link
Contributor Author

missytake commented Aug 27, 2025

I rebased this PR on main, because #617 is controversial - so now we can review and merge this independently :)

@missytake missytake force-pushed the upload-custom-www-529 branch from 3bf441e to ef1c3d2 Compare August 29, 2025 21:49
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.

(Option to) avoid building/deploying webpages
1 participant