Skip to content

Add default Derivation Group associations for Plans at the Model level#1831

Open
pranav-super wants to merge 3 commits into
developfrom
feature/model_derivation_group_associations_v2
Open

Add default Derivation Group associations for Plans at the Model level#1831
pranav-super wants to merge 3 commits into
developfrom
feature/model_derivation_group_associations_v2

Conversation

@pranav-super
Copy link
Copy Markdown
Contributor

@pranav-super pranav-super commented May 20, 2026

  • Tickets addressed: N/A
  • Review: By commit
  • Merge strategy: Merge (no squash)

Description

During the exploration of PlanDev by operators, it was found that manual association of derivation groups on new plan creation was rather tedious, as the same derivation groups were repeatedly being associated with each new plan.

This PR suggests a QoL fix, in tandem with this PR on plandev-ui, to allow derivation group association defaults to be set up at a model level. This is much like the default association specifications that are possible for goals, constraints, and conditions, and is modeled after that functionality.

Verification

No new tests were added, but the changes were tested in the UI, which require that these few changes on the backend are correct and functional.

Documentation

No updates to documentation should be necessary.

Future work

No future work is anticipated with regards to this update specifically.

@pranav-super pranav-super requested a review from a team as a code owner May 20, 2026 23:15
@pranav-super pranav-super force-pushed the feature/model_derivation_group_associations_v2 branch from 2e27296 to b6f7599 Compare May 20, 2026 23:35
Copy link
Copy Markdown
Contributor

@Mythicaeda Mythicaeda left a comment

Choose a reason for hiding this comment

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

LGTM! However, please hold off on merging until the UI PR is ready

@Mythicaeda Mythicaeda added the DON'T MERGE Do Not Merge This Branch label May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

database Anything related to the database DON'T MERGE Do Not Merge This Branch feature A new feature or feature request Waiting for UI This PR should not be merged until its sibling UI PR is ready to merge.

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants