Skip to content

Comments

fix: small improvements#1

Merged
festo merged 3 commits intomainfrom
improve-and-fix-errors
Jan 26, 2026
Merged

fix: small improvements#1
festo merged 3 commits intomainfrom
improve-and-fix-errors

Conversation

@festo
Copy link
Contributor

@festo festo commented Jan 26, 2026

Description of change

  • Fix the GET cats/:id endpoint, the path parameter was ignored
  • Fix the skipped test. By default typeORM does the binary<>uuid conversion, but for raw queries, it must be done manually. For this reason, UUID_VALUE_TRANSFORMER is exported.
  • Make the params optional for BinaryUuidColumn so it can be used like BinaryUuidColumn()
  • Fix ApiPaginatedResponse to return a correct OpenAPI type. With the previous allOf the generated typescript interface was something like this: items: string[] & ContentItem[], now it is just items: ContentItem[] as it should be

Pull-Request Checklist

  • Code is up-to-date with the main branch
  • npm run lint passes with this change
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #0000 - N/A
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change
  • The new commits follow conventions outlined in the conventional commit spec

@festo festo requested a review from dstockhammer January 26, 2026 14:14
@festo
Copy link
Contributor Author

festo commented Jan 26, 2026

TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".mts" for /home/runner/work/nestjs-utils/nestjs-utils/prettier.config.mts

Strange error, I did not change the configuration

Copy link
Member

Choose a reason for hiding this comment

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

I think you can delete this test - query by id should be enough; I just added this one because I couldn't get the id to work.

@festo festo merged commit a4677ca into main Jan 26, 2026
5 checks passed
@github-actions
Copy link

🎉 This PR is included in version 1.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants