Skip to content

FOUR-33485: Launch Screen is not updated on processes page - #9079

Open
henryjonathanquispe wants to merge 6 commits into
developfrom
FOUR-33485
Open

henryjonathanquispe wants to merge 6 commits into
developfrom
FOUR-33485

Conversation

@henryjonathanquispe

Copy link
Copy Markdown
Contributor

Related Tickets & Packages

Code Review Checklist

  • I have pulled this code locally and tested it on my instance, along with any associated packages.
  • This code adheres to ProcessMaker Coding Guidelines.
  • This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
  • This solution fixes the bug reported in the original ticket.
  • This solution does not alter the expected output of a component in a way that would break existing Processes.
  • This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
  • This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
  • This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
  • This ticket conforms to the PRD associated with this part of ProcessMaker.

ci:deploy

@cursor

cursor Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Change is limited to the processes catalogue JS entry point and aligns bootstrap with the Screen Builder package; main caveat is skipping screenBuilderNext cache/script setup if that page relied on it.

Overview
Fixes FOUR-33485 by changing how Screen Builder is bootstrapped on the processes catalogue entry (loaderProcessesCatalogue.js).

The loader no longer calls the local screenBuilderNext() wrapper. It now imports @processmaker/screen-builder (and vue-form-builder.css) and registers the plugin with window.Vue.use(ScreenBuilder.default) after setupMain(), so launch screens on the processes page pick up the packaged builder instead of staying on the deferred/async registration path.

Reviewed by Cursor Bugbot for commit 498bbcc. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 96a557d. Configure here.

Comment thread resources/js/processes-catalogue/loaderProcessesCatalogue.js
@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://tenant-1.ci-6f7c86008b.engk8s.processmaker.net

Removed import for vue-form-builder.css from loaderProcessesCatalogue.js
@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://tenant-1.ci-6f7c86008b.engk8s.processmaker.net

@pmPaulis pmPaulis added 2026.16 and removed 2026.15 labels Sep 29, 2026
@decisions-sonarqube

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://tenant-1.ci-6f7c86008b.engk8s.processmaker.net

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants