Skip to content

Conversation

@eiresendez
Copy link
Contributor

@eiresendez eiresendez commented Jan 15, 2026

Issue & Reproduction Steps

  1. Open a Screen Builder screen with a Record List configured with a Collection.
  2. Click “Open Edit Screen” on the Record List inspector.
  3. It navigates to /designer/screen-builder/null/edit even though the collection has screens assigned.

Solution

  • Recalculate collection screen IDs after collections are loaded so the link is built with a real screen ID when the collection is already selected.

How to Test

  1. Open the Screen Builder for a screen containing a Record List linked to a Collection.
  2. Verify “Open Edit Screen” opens the correct screen (not null).
  3. Change the collection and verify the link updates correctly.

Related Tickets & Packages

ci:deploy

@cypress
Copy link

cypress bot commented Jan 15, 2026

screen-builder    Run #2179

Run Properties:  status check passed Passed #2179  •  git commit f31266a914: FOUR-25350: Fix record list screen links when collections load asynchronously
Project screen-builder
Branch Review bugfix/FOUR-25350
Run status status check passed Passed #2179
Run duration 09m 42s
Commit git commit f31266a914: FOUR-25350: Fix record list screen links when collections load asynchronously
Committer Eleazar Resendez
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 19
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 389
View all changes introduced in this branch ↗︎

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-69e44c364f.engk8s.processmaker.net

@processmaker-sonarqube
Copy link

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

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.

4 participants