-
Notifications
You must be signed in to change notification settings - Fork 14
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
Feature/new change brand status endpoint #7
base: main
Are you sure you want to change the base?
Feature/new change brand status endpoint #7
Conversation
…g incorrect MongoFilters typing
import {MongoPipelineStage} from "./MongoPipeline"; | ||
|
||
export type MongoMatchValueType = string | number | Date | object; | ||
export type MongoFilters = {$and: AnyObject}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to fix this, because I was getting a warning
-- | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes from this file are:
- Removing unnecessary quotes.
- Adding the new
status
column.
@@ -26,6 +26,7 @@ | |||
"@aws/dynamodb-data-mapper": "^0.7.3", | |||
"@aws/dynamodb-data-mapper-annotations": "^0.7.3", | |||
"@graphql-tools/schema": "^8.5.1", | |||
"@prisma/client": "^5.3.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This dependency was missing
Summary
@prisma/client
dependency.MongoMatch.ts
IN_PROGRESS
as default value).DS_Store
to.gitignore
.BrandStatus
as a column to the Brand Entity, to the GraphQL Schema and therefore to the existing GraphQL queries.BrandInput
, changeupdateInput()
function and test it.updateBrand()
mutation by adding a newstatus
attribute, and that's it.