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

Issue #132: Added Module for Product Images #143

Merged
merged 9 commits into from
Feb 12, 2025

Conversation

Shukazuby
Copy link
Contributor

@Shukazuby Shukazuby commented Feb 11, 2025

Issue #132: Added Module for Product Images

Closes #132

Added Service, Resolver and Module to add, fetch all, and fetch product images by Id.

To help with the task, I created a provisional product module to create product, reference it to product image and tested with created product id.

productImage productImages

@derianrddev derianrddev added enhancement New feature or request backend labels Feb 11, 2025
@derianrddev derianrddev self-requested a review February 11, 2025 15:08
@derianrddev
Copy link
Contributor

@Shukazuby Could you please share a screenshot of GraphQL queries for ProductImage, similar to this one? It can be with any test data, I just need an example that includes fetching all product images and a specific product image by ID.
image

@derianrddev
Copy link
Contributor

@Shukazuby Once you finish, please run npm run lint:backend to format the code and resolve this linter error.

image

Copy link
Contributor

@derianrddev derianrddev left a comment

Choose a reason for hiding this comment

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

@Shukazuby It looks like a change hasn’t been addressed yet. Please make sure to resolve it and add the migration, please.

apps/backend/prisma/schema.prisma Show resolved Hide resolved
@derianrddev derianrddev self-requested a review February 11, 2025 18:49
Copy link
Contributor

@derianrddev derianrddev left a comment

Choose a reason for hiding this comment

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

@Shukazuby Thank you for making the changes. LGTM! Great job!🚀Let’s wait for a second review. @danielcdz

@danielcdz danielcdz merged commit 78b5d8d into KaizeNodeLabs:main Feb 12, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT]: Add Module for Product Images
3 participants