Skip to content

Commit 85e40cf

Browse files
committed
fix: don't skip app builds even if components haven't changed
- renderer could have changed
1 parent 55712eb commit 85e40cf

File tree

2 files changed

+1
-7
lines changed

2 files changed

+1
-7
lines changed

frontend/src/components/DynamicValueSelector.vue

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@ const emit = defineEmits<{
6161
}>()
6262
const bindVariable = ref(!!props.isVariableBound)
6363
64-
// Watch for external changes to isVariableBound prop
6564
watch(
6665
() => props.isVariableBound,
6766
(newValue) => {

studio/studio/doctype/studio_app/studio_app.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -139,13 +139,8 @@ def generate_app_build(self):
139139
if not frappe.has_permission("Studio App", ptype="write"):
140140
frappe.throw("You do not have permission to generate the app build", frappe.PermissionError)
141141

142-
# check if build is required
143-
draft_components = get_app_components(self.name, "draft_blocks")
144-
components = get_app_components(self.name)
145-
if not draft_components.symmetric_difference(components):
146-
return
147-
148142
try:
143+
components = get_app_components(self.name)
149144
command = f"yarn build-studio-app {self.name} --components {','.join(list(components))}"
150145
studio_app_path = frappe.get_app_source_path("studio")
151146
popen(command, cwd=studio_app_path, raise_err=True)

0 commit comments

Comments
 (0)