Should not keep commented out code.
Should not keep commented out code.
Should not keep commented out code.
Should not keep commented out code.
Should not keep commented out code.
Is this using necessary to be added?
f1e04396de
Merge pull request 'B2026-042-Fix-Null-Error-for-Set_At_All_Level-in-the-Applicability-tab-evaluate-having-the-button-grayed-out-for-the-RNO-column-4' (#762) from B2026-042-Fix-Null-Error-for-Set_At_All_Level-in-the-Applicability-tab-evaluate-having-the-button-grayed-out-for-the-RNO-column-4 into Development
d0cae8c2fc
B2026-042-Fix-Null-Error-for-Set_At_All_Level-in-the-Applicability-tab-evaluate-having-the-button-grayed-out-for-the-RNO-column-4
90c215d4c1
B2026-042-Fix-Null-Error-for-Set_At_All_Level-in-the-Applicability-tab-evaluate-having-the-button-grayed-out-for-the-RNO-column-4
1b1dc78548
C2026-027-New-Set_At_All_Level-in-the-Applicability-tab-evaluate-having-the-button-grayed-out-for-the-RNO-column-4
With latest change, looks good to me. Ready for QA!
Please let me know if you have any questions - I did a screenshot of the code to re-illustrate.
As mentioned above this 2nd if statement that you added is unnecessary - I took a screenshot and attached here to re-illustrate. The part with a purple square is unnecessary because that is…
Also, I am not sure how this would ever get hit as isn't this basically the same check as line 405?
With both if statements --- if it is null, do we want it enabled or not?
Looks good to me. Ready for QA.
Also in the designer --- Just to check --- is this wording accurate? ----- i.e. if this is ran on a high level step wouldn't it update all high level steps in that procedure (not sub-steps)?