Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

observation/FOUR-20173 The open screen of the Collection Record control opens a different screen #1776

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

gustavobascope
Copy link
Contributor

@gustavobascope gustavobascope commented Nov 14, 2024

Issue & Reproduction Steps

The open screen of the Collection Record control opens a different screen

Expected behavior:
When you click on open screen of the collection record control, it should open the screen that is in the collection

Actual behavior:
As we can see, when clicking on open screen of the collection record screen control, it opens a different screen than the one configured in the collection.

Solution

Added a method to obtain the screen id for editing collections

How to Test

Log in
Go to Designer
Click on Screens
Create a new form screen
Create a Collection Record Control
Configure. with collection
Click on Opone screen

Related Tickets & Packages

https://processmaker.atlassian.net/browse/FOUR-20173

Copy link

cypress bot commented Nov 14, 2024

screen-builder    Run #1693

Run Properties:  status check passed Passed #1693  •  git commit 36160d3625: observation/FOUR-20173 The open screen of the Collection Record control opens a ...
Project screen-builder
Branch Review observation/FOUR-20173
Run status status check passed Passed #1693
Run duration 09m 23s
Commit git commit 36160d3625: observation/FOUR-20173 The open screen of the Collection Record control opens a ...
Committer gustavobascope
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 365
View all changes introduced in this branch ↗︎

Copy link
Contributor

@CarliPinell CarliPinell left a comment

Choose a reason for hiding this comment

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

@gustavobascope Fix is working fine.
Approved.

@tokensPM
Copy link

QA server K8S was successfully deployed https://ci-8dff3d5cf4.engk8s.processmaker.net

@ryancooley ryancooley merged commit 6dcc76b into release-2024-fall Nov 20, 2024
41 checks passed
@ryancooley ryancooley deleted the observation/FOUR-20173 branch November 20, 2024 14:52
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