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

feat: updated typegoose 9.x dependency #465

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dereekb
Copy link

@dereekb dereekb commented Feb 16, 2022

  • Updated typegoose dependency to 9.x. Uses mongoose 6, mongodb 5, etc.
  • Updated nestjs support to 8.x

Should resolve:

#460 #452 #433

- Updated typegoose dependency to 9.0
- mongoose 6 support / requirement
@dereekb
Copy link
Author

dereekb commented Feb 16, 2022

The typings didn't change too much. Luckily this library just passes through the typings instead of uses them extensively and got to avoid all the mongoose typings shenanigans like typegoose's library encountered.

Only the tests had some minor changes needed to reflect the new connection options, and updates to mongodb-memory-server.

Nestjs 8 didn't have any breaking changes. I went ahead and just bumped the version to 8, but the nestjs dependencies could probably remain 6 | 7 | 8.

@Nierifeng
Copy link

Can you send an npm package?

@dereekb
Copy link
Author

dereekb commented Apr 18, 2022

Can you send an npm package?

You can replace your "nestjs-typegoose" entry with this in your package.json to pull from my branch that has the build/dist available within it.

"nestjs-typegoose": "git+https://[email protected]/dereekb/nestjs-typegoose.git#ae3da62f6c26ec9ad3df3f69b31dd2db84327c65",

@Gallahar
Copy link

Can you send an npm package?

You can replace your "nestjs-typegoose" entry with this in your package.json to pull from my branch that has the build/dist available within it.

"nestjs-typegoose": "git+https://[email protected]/dereekb/nestjs-typegoose.git#ae3da62f6c26ec9ad3df3f69b31dd2db84327c65",

Good day sir, seem's its outdated too ;(

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.

5 participants