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

Support the new 1.x pgmodeler version #108

Open
artis3n opened this issue Feb 2, 2023 · 5 comments · May be fixed by #119
Open

Support the new 1.x pgmodeler version #108

artis3n opened this issue Feb 2, 2023 · 5 comments · May be fixed by #119
Assignees

Comments

@artis3n
Copy link
Owner

artis3n commented Feb 2, 2023

Uses QT6 instead of QT5, explore other changes.

@danielarantes-novonix
Copy link

Hey, I did some work on this because the previous version wasnt connecting to my pgsql instance (for whatever reason).
I will do some tidying up (probably don't need all the qt6 related libs I added) and put a PR for you.
Great work on this!

@artis3n
Copy link
Owner Author

artis3n commented Dec 13, 2023

Thanks! That'll be much appreciated. I've been meaning to come back to this and review the 1.x changes and get things compatible again

@danielarantes-novonix
Copy link

I am not familiar with git submodules, so when I updated it got the v1.1.0-beta instead of the stable one. There might be a way to control what branch and tag to pull from.

@artis3n
Copy link
Owner Author

artis3n commented Dec 13, 2023

I like this explanation, but yes it's all a bit overcomplicated: https://stackoverflow.com/a/67796692

@artis3n artis3n self-assigned this Dec 22, 2023
@artis3n artis3n linked a pull request Dec 22, 2023 that will close this issue
@artis3n
Copy link
Owner Author

artis3n commented Feb 2, 2024

Working on this in #119 , have to resolve QT6 errors. The submodules are updated already in that PR.

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 a pull request may close this issue.

2 participants