-
Notifications
You must be signed in to change notification settings - Fork 24
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/master order application list #1063
Conversation
…; Add all missing joins to application list query
…master_order-application-list
Add fix for linter issues
Deploy preview for clever-edison-cd22c1 ready! Built with commit 1868f93 https://deploy-preview-1063--clever-edison-cd22c1.netlify.app |
@IsOptional({ groups: [ValidationsGroupsEnum.default] }) | ||
@IsString({ groups: [ValidationsGroupsEnum.default] }) | ||
@IsIn(Object.keys(OrderParamEnum), { groups: [ValidationsGroupsEnum.default] }) | ||
@Transform((value: string | undefined) => (value ? value : OrderParamEnum.DESC), { |
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.
Is toClassOnly
required here since this is only ever used for input?
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.
Fixed
import { PaginationQueryParams } from "../shared/dto/pagination.dto" | ||
import { ValidationsGroupsEnum } from "../shared/validations-groups.enum" | ||
import { defaultValidationPipeOptions } from "../shared/default-validation-pipe-options" | ||
import { applicationPreferenceExtraModels } from "./entities/application-preferences.entity" | ||
import { ApplicationCsvExporter } from "../csv/application-csv-exporter" | ||
|
||
enum OrderByParamEnum { |
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.
So far we have not been adding Enum suffix to the enums
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.
Fixed
Removed toClassOnly from Transform decorator options Removed Enum suffix from OrderByParamEnum and OrderParamEnum
Related to: #866