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

fix: incorrect mongoose type used for ConnectOptions #1101

Merged
merged 1 commit into from
May 13, 2024

Conversation

FTheron
Copy link
Contributor

@FTheron FTheron commented May 7, 2024

Definition of Done

Building forest-admin projects with typescript should not fail.
Error: '"mongoose"' has no exported member named 'ConnectionOptions'. Did you mean 'ConnectOptions'?

General

This is a typing issue I detected when using typescript in our builds. Mongoose does not export typeConnectionOptions. It should be ConnectOptions.

Security

No impact

PS: Adding "skipLibCheck": true in our tsconfig works as a temporary workaround.

@FTheron FTheron changed the title fix: ConnectOptions mongoose type fix: typo in mongoose type May 7, 2024
@FTheron FTheron marked this pull request as ready for review May 7, 2024 12:11
@FTheron FTheron changed the title fix: typo in mongoose type fix: incorrect mongoose type used for ConnectOptions May 7, 2024
@ghusse
Copy link
Contributor

ghusse commented May 7, 2024

Can you amend the commit to use the same message you used in the PR title? The current CI is not ideal, because it checks every commit to verify they're following the conventional commit convention.

This will allow us to run the CI on your PR.

@ghusse ghusse merged commit 290276a into ForestAdmin:main May 13, 2024
3 of 4 checks passed
forest-bot added a commit that referenced this pull request May 13, 2024
## [9.3.22](v9.3.21...v9.3.22) (2024-05-13)

### Bug Fixes

* incorrect mongoose type used for ConnectOptions ([#1101](#1101)) ([290276a](290276a))
@forest-bot
Copy link
Member

🎉 This PR is included in version 9.3.22 🎉

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.

4 participants