Skip to content

Conversation

@patrickseguraoddball
Copy link
Contributor

@patrickseguraoddball patrickseguraoddball commented Sep 11, 2025

MINT-3195

Description

This is a high-level feature review of MTO template. All technical requirements have been review, approved, and merged into the feature. You may still do a technical review, but not a hard requirement

How to test this change

  1. Thoroughly test all requirements functionally and designs match figma

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.

Peyton Thibodeaux and others added 18 commits September 9, 2025 09:14
* added migrations for mto template tables

* changed common solution reference from key to id

* requested migration changes

* changed enum to be template keys

---------

Co-authored-by: Peyton Thibodeaux <peytonthibodeaux@Peytons-MacBook-Pro.local>
* added gql changes for mto template library

* added mock returns and query changes

* added mock milestone, category and solutions

* modfied resolvers to be nested like MTO

* added solutions to top level mto template

* fixed bruno calls for mto templates

* added milestone to top leval also

---------

Co-authored-by: Peyton Thibodeaux <peytonthibodeaux@Peytons-MacBook-Pro.local>
Copilot AI review requested due to automatic review settings September 11, 2025 14:55
@patrickseguraoddball patrickseguraoddball requested review from Peyton232 and removed request for a team September 11, 2025 14:55
@patrickseguraoddball patrickseguraoddball added the Feature Branch Used to track PRs that track feature branches going into main label Sep 11, 2025
@patrickseguraoddball patrickseguraoddball requested review from yiranli624 and removed request for a team September 11, 2025 14:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a new MTO (Models to Operations) Template feature that provides reusable template structures for organizing model plans with predefined categories, milestones, and solutions.

  • Database schema with template tables, categories, milestones, and solutions
  • GraphQL API for querying templates and applying them to model plans
  • Mock resolvers with sample template data for testing

Reviewed Changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
migrations/V224__Add_MTO_Template_Audit.sql Adds audit table type enums for MTO template tables
migrations/V225__Add_MTO_Template_tables.sql Creates complete database schema with tables, constraints, and triggers
pkg/models/mto_template.go Defines Go models and constructors for MTO template entities
pkg/graph/schema/types/mto_template.graphql GraphQL schema definitions for template types and operations
pkg/graph/resolvers/mto_template.resolvers.go Mock GraphQL resolvers for template queries and mutations
pkg/graph/model/models_gen.go Generated GraphQL model for ApplyTemplateResult
query_examples/MINT/Models to Operations Matrix/Templates/*.bru Bruno API collection files for testing template endpoints

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Branch Used to track PRs that track feature branches going into main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants