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.

Sign up for GitHub

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

Jump to bottom

[MINT-3175] Models to Operation Matrix (MTO) Feature #1443

Draft
wants to merge 753 commits into
base: main
Choose a base branch
from
Draft

[MINT-3175] Models to Operation Matrix (MTO) Feature #1443

wants to merge 753 commits into from

Conversation

ClayBenson94
Copy link
Collaborator

@ClayBenson94 ClayBenson94 commented Oct 24, 2024 •
edited
Loading

MINT-3175

Note

The Figma mockups for this feature are available here!

Tip

To see a list of PRs that targeted this feature branch, click this link!

Description

  • TODO

How to test this change

  1. TODO

PR Author Checklist

  • I have provided a detailed description of the changes in this PR.
  • I have provided clear instructions on how to test the changes in this PR.
  • I have updated tests or written new tests as appropriate in this PR.
  • Updated the Postman Collection if necessary.

PR Reviewer Guidelines

  • It's best to pull the branch locally and test it, rather than just looking at the code online!
  • When approving a PR, provide a reason why you're approving it
    • e.g. "Approving because I tested it locally and all functionality works as expected"
    • e.g. "Approving because the change is simple and matches the Figma design"
  • Don't be afraid to leave comments or ask questions, especially if you don't understand why something was done! (This is often a great time to suggest code comments or documentation updates)
  • Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.

@ClayBenson94 ClayBenson94 added the Feature Branch Used to track PRs that track feature branches going into main label Oct 24, 2024
@OddTomBrooks @StevenWadeOddball
[MINT-3226] Edit Solutions Linked to Milestones (#1577)
8561d48 
* feat: support for editing milestone linked solutions

* chore: updated postman collection

* chore: updated gql, adding solution linking to update mutation

* feat: first pass batch allow conflict insert

* feat: attempt to make a batch allow insert common solutions and merge solutions to milestones

* feat: first pass at upsert and linking for solutions / milestones

* wip: returning solutions, off-by-one return value set error

* feat: correct some syntax, pq complains about matched BY source thouhg

* feat: version with functioning merge, though no return

* feat: insert common solutions allow conflicts in batched manner

* chore: updated postman collection and fixed transaction

* feat: first pass returning solutions instead of links and inserting and deleting with the logic from set_solution_links instead

* feat: store actor session variable for the merge statement

* feat: wrapper function around transactions

* feat: leverage shared update links resolver in the milestone update resolver

* feat: mto milestone create common leverages the shared merge logic

* feat: remove unused set solution links

* feat: coalesce to correctly return name and type from common solution table

* feat: tx --> np for milestone update

---------

Co-authored-by: Steven Wade <[email protected]>
Co-authored-by: Steven Wade <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Feature Branch Used to track PRs that track feature branches going into main
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ClayBenson94 @garyjzhao @patrickseguraoddball @StevenWadeOddball @OddTomBrooks