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

Cannot find module './../type-helpers/set-required' #12109

Closed
2 of 7 tasks
ynhhoJ opened this issue Apr 15, 2020 · 1 comment
Closed
2 of 7 tasks

Cannot find module './../type-helpers/set-required' #12109

ynhhoJ opened this issue Apr 15, 2020 · 1 comment

Comments

@ynhhoJ
Copy link

ynhhoJ commented Apr 15, 2020

Issue Description

It seems bug from: #11973 is not resolved yet.

What are you doing?

I'm using typescript with sequelize v6.0.0-beta.5.
Code:

import { Sequelize } from 'sequelize';

const sequelize = new Sequelize('db', 'username', 'password', {
    host: 'localhost',
    dialect: 'postgres',
    pool: {
        max: 5,
        min: 0,
        acquire: 30000,
        idle: 10000
    }
});

sequelize
    .authenticate()
    .then(() => {
        console.log('Connection has been established successfully.');
    })
    .catch((err: string) => {
        console.error('Unable to connect to the database:', err);
    });

export default sequelize;

What do you expect to happen?

Resolve error.

What is actually happening?

On tsc --noEmit i get this error:

node_modules/sequelize/types/lib/query-interface.d.ts(7,29): error TS2307: Cannot find module './../type-helpers/set-required'

Environment

  • Sequelize version: 6.0.0-beta.5
  • Node.js version: 12.8.0
  • Operating System: Windows 10, v1909
  • If TypeScript related: TypeScript version: 3.6.3

Issue Template Checklist

How does this problem relate to dialects?

  • I think this problem happens regardless of the dialect.
  • I think this problem happens only for the following dialect(s):
  • I don't know, I was using PUT-YOUR-DIALECT-HERE, with connector library version XXX and database version XXX

Would you be willing to resolve this issue by submitting a Pull Request?

  • Yes, I have the time and I know how to start.
  • Yes, I have the time but I don't know how to start, I would need guidance.
  • No, I don't have the time, although I believe I could do it if I had the time...
  • No, I don't have the time and I wouldn't even know how to start.
@papb
Copy link
Member

papb commented Apr 16, 2020

Hello, in fact there has been no version release yet, PR #12000 was merged after v6.0.0-beta.5.

If you need this asap, you can install directly from the github commit tarball:

npm install https://github.com/sequelize/sequelize/tarball/9785c585de010dca015b345dab9caa6b27f952a3

Thanks!

@papb papb closed this as completed Apr 16, 2020
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

No branches or pull requests

2 participants