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

#3166 Created Part/PartPreview Shared Types and Relevant Data #3181

Draft
wants to merge 1 commit into
base: feature/cad-project-file-review
Choose a base branch
from

Conversation

justalittlemushroom
Copy link
Contributor

@justalittlemushroom justalittlemushroom commented Feb 2, 2025

Changes

Created Part, PartPreview, PartTag, PartSubmission, PartReview, and PartReviewPopUp interfaces

To Do

  • There is a parts-review.query-args.ts file with query args for part-preview, which returns the model with all relevant information within a part, including submissions and reviews
  • There is a transformer in parts-review.transformer.ts that converts a prisma model type into the part shared type
  • There is a shared type in parts-review.types.ts in the shared folder with corresponding interfaces and relevant data

Checklist

  • All commits are tagged with the ticket number
  • No linting errors / newline at end of file warnings
  • All code follows repository-configured prettier formatting
  • No merge conflicts
  • All checks passing
  • Screenshots of UI changes (see Screenshots section)
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • No yarn.lock changes (unless dependencies have changed)
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Sorry, something went wrong.

@justalittlemushroom justalittlemushroom self-assigned this Feb 2, 2025
@justalittlemushroom justalittlemushroom marked this pull request as draft February 17, 2025 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant