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

Add initial support for Sequelize V6 #223

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

Add initial support for Sequelize V6 #223

wants to merge 1 commit into from

Conversation

aaronovz1
Copy link

Hacked up a quick solution for V6 support. The tests pass for mysql but I did not test the other types.

#174

@nicholasgriffintn
Copy link

This works great for v6, just tried it on my own private fork and this indeed allowed me to upgrade to v6.

@aaronovz1
Copy link
Author

This works great for v6, just tried it on my own private fork and this indeed allowed me to upgrade to v6.

Glad I could help 👍 It does seem like project is not maintained so probably best to maintain your own fork like I did.

@nicholasgriffintn
Copy link

Oh massively, saved me a couple of hours probably, but yes a private fork is probably best, going to look at possible upgrades while I have one see if there's any other stuff I can extend this with.

@rgwebcode
Copy link

Any news?
It seems all the checks above failed because it couldn't connect to the MySQL server in the first place, so this was not particularly helpful.

@nicholasgriffintn
Copy link

The code works! I just think this repo is dead unfortunately. I’d suggest forking it and personally publishing it, or making a community fork.

I did a org private fork so I can’t share mine.

@overlookmotel
Copy link
Owner

Yeah, I'm sorry I haven't found any time to work on this in past year or two. I don't use it myself any more, so it's hard to push up list of priorities, but it's a shame to see what has been I think useful to many people sliding into disrepair.

I've opened an issue #234 asking if anyone wants to step up and take over as maintainer. Perhaps this would be a good fit for one of those who has already produced a fork?

@jefersonlucio
Copy link

And what became of it?

@OhadR
Copy link

OhadR commented Feb 4, 2024

thanks!
i would even remove bluebird and work with node's Promise...

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.

7 participants