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

chore. Compatibility with the latest version #24

Merged
merged 15 commits into from
Mar 20, 2024

Conversation

pangolp
Copy link
Contributor

@pangolp pangolp commented Mar 17, 2024

I have not tested the modules individually. I was only in charge of updating and correcting compilation errors in Windows at least. Now we will see if it passes the tests in Linux as well, and if something needs to be modified, it will be accommodated. There may still be things to do, but perhaps after this, it will be easier to find and correct.

@pangolp
Copy link
Contributor Author

pangolp commented Mar 17, 2024

I didn't know that the SQL was also outdated. This is going to take a lot longer than she thought. It would have been better to work on the modules separately than to put them all together, even if they had a relationship or requirements between them, it could have been clarified and that was it. This way, it will be more difficult to update, it seems to me.

@pangolp
Copy link
Contributor Author

pangolp commented Mar 17, 2024

This error, I don't know how to correct it.
The last commit does not solve the problem.

@pangolp
Copy link
Contributor Author

pangolp commented Mar 19, 2024

Thanks @Helias for advice.

@pangolp
Copy link
Contributor Author

pangolp commented Mar 19, 2024

I'm tired, I'm going to build a machine with docker, because that way I can fix all the bugs together, if not, I'm going to make 200 commits at this rate.

@pangolp pangolp merged commit 9c25df7 into azerothcore:master Mar 20, 2024
1 check failed
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.

Massive Build Fail Missing DB Table AutoBalance.h Compile Error Fixed Error Compiled
2 participants