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

Procedural Constraints UI #1537

Open
wants to merge 13 commits into
base: develop
Choose a base branch
from
Open

Conversation

duranb
Copy link
Collaborator

@duranb duranb commented Oct 31, 2024

resolves #1478

___REQUIRES_AERIE_PR___="1596"

To test:

  1. Create a constraint, but click the new "JAR" button
  2. Select a JAR file (any .jar works for showcasing the UI functionality)
  3. Finish creating the constraint
  4. Open/create a plan
  5. Open the "Constraints" panel
  6. Click the "Manage Constraints" button
  7. Select the newly created constraint and click the "Update" button
  8. Verify that the constraint shows up in the Constraints panel
  9. Right click the constraint name
  10. Click "Duplicate Invocation"
  11. Verify that another entry with the same constraint name shows up
  12. Right click the new constraint name
  13. Click "Delete Invocation"
  14. Verify that the instance is deleted

TODO:

  • Add constraint invocation argument diffing in order to determine if the displayed status is still relevant

@duranb duranb force-pushed the feature/procedural-constraints branch from 926a0dd to 1a5b894 Compare October 31, 2024 18:04
@duranb duranb force-pushed the feature/procedural-constraints branch from 49d007d to d0930e3 Compare November 4, 2024 22:58
@duranb duranb force-pushed the feature/procedural-constraints branch from 89ede2d to 26d639c Compare November 6, 2024 19:20
@duranb duranb force-pushed the feature/procedural-constraints branch from bd3d47e to 5a56e25 Compare November 19, 2024 16:58
@duranb duranb force-pushed the feature/procedural-constraints branch from 5a56e25 to fc5d1a1 Compare December 4, 2024 21:44
@duranb duranb force-pushed the feature/procedural-constraints branch from bf14ee8 to ba9cccc Compare December 6, 2024 17:16
@duranb duranb force-pushed the feature/procedural-constraints branch from ba9cccc to 4a39f89 Compare December 6, 2024 17:16
@duranb duranb marked this pull request as ready for review December 6, 2024 17:16
@duranb duranb requested a review from a team as a code owner December 6, 2024 17:16
@duranb duranb added the DON'T MERGE Do Not Merge This Branch label Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DON'T MERGE Do Not Merge This Branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UI updates for Procedural Constraints
1 participant