Skip to content

Update swagger decorator Json type mapping#70

Open
ashenm wants to merge 1 commit into
kimjbstar:mainfrom
ashenm:swagger-json-type
Open

Update swagger decorator Json type mapping#70
ashenm wants to merge 1 commit into
kimjbstar:mainfrom
ashenm:swagger-json-type

Conversation

@ashenm
Copy link
Copy Markdown
Contributor

@ashenm ashenm commented Aug 12, 2024

Fixes

unexpected error occured
SyntaxError: '(' expected. (27:46)
  25 |  external_id: string 
  26 | 
> 27 |  @ApiProperty({type: Record<string, unknown> | unknown[] | boolean | string | number | null})
     |                                              ^
  28 |  metadata: Record<string, unknown> | unknown[] | boolean | string | number | null 
  29 | 
  30 | }

Proposed Changes

  • For swagger decorator, use Object for type Json instead of typescript types

@ashenm
Copy link
Copy Markdown
Contributor Author

ashenm commented Aug 12, 2024

@kimjbstar at your convenience 🙏

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.

1 participant