Skip to content

feat: Complete the cycle for custom & standard Studio App builds#163

Open
ruchamahabal wants to merge 8 commits intodevelopfrom
build
Open

feat: Complete the cycle for custom & standard Studio App builds#163
ruchamahabal wants to merge 8 commits intodevelopfrom
build

Conversation

@ruchamahabal
Copy link
Copy Markdown
Member

@ruchamahabal ruchamahabal commented Apr 6, 2026

Before

App-specific builds existed & studio used to only generate these on hitting publish. But the cycle was not complete.

Current State:

  • All app builds go to studio/public/app_builds/{app_name}/ regardless of standard vs custom
  • That public/ folder is symlinked to sites/assets/studio/ locally, but FC doesn't symlink — it copies assets/ once during deploy
  • There is no automatic rebuild when renderer changes (after bench build or bench migrate)
  • The get_app_components function reads from DB (frappe.get_all("Studio Page", ...)), which doesn't work in a no-site bench build context for standard apps
  • Standard apps exported to other frappe apps (e.g., frappe_app = "my_app") still write builds to studio/public/ instead of my_app/public/

After

There are 2 types of apps one can create from Studio

  • Standard apps (exported to files)
  • Custom apps (stored in site DB)

Completing the build cycle to handle both these categories of builds

Studio App Builds

Closes #144, Closes #97

@ruchamahabal ruchamahabal changed the title fix: app build process fix: app builds Apr 6, 2026
standard app: `{frappe_app}/public/app_builds`, custom apps: `sites/{sitename}/files/app_builds`

- also hook exported studio app builds with frappe builds & custom studio app builds after migrate
- was using a relative path and generating builds in studio/frontend itself, use abs path instead
@ruchamahabal ruchamahabal changed the title fix: app builds feat: Complete the cycle for custom & standard Studio App builds Apr 9, 2026
@ruchamahabal ruchamahabal marked this pull request as ready for review April 9, 2026 13:55
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.

Publisher get blank page App Publish Error

1 participant