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

Added DBML Serializer (just hacked it dirty in, better than nothing) #159

Merged
merged 15 commits into from
Oct 28, 2023

Conversation

Kaev
Copy link
Contributor

@Kaev Kaev commented Oct 27, 2023

Quick and dirty implementation for DBML export to generate this: https://dbdocs.io/Kaev/DBC_3_3_5_12340
Only exports for a single build that you need to provide as a new argument.

bloerwald and others added 8 commits October 27, 2023 23:19
First there was only TerrainID, the primary key. Then due to the non-"ID" name a new column was auto-generated. At some point someone noticed that and the IDs got made equivalent to TerrainID. It is unknown whether the column is now called TerrainID or ID, but TerrainID values are consistent since Vanilla with *this* naming scheme.
@Marlamin
Copy link
Collaborator

Not sure why there's a bunch of Structs. related changes in other files unrelated to this functionality, seems unnecessary, don't think it'll break anything but might be better to just not do those.

@Kaev
Copy link
Contributor Author

Kaev commented Oct 28, 2023

Not sure why there's a bunch of Structs. related changes in other files unrelated to this functionality, seems unnecessary, don't think it'll break anything but might be better to just not do those.

Rider replaced that for me when i did the initial (reverted) namespace change. I reverted these. :)

@Kaev
Copy link
Contributor Author

Kaev commented Oct 28, 2023

I merged #157 and #158 into this so only merge after they got merged

@bloerwald bloerwald merged commit 8f3fe27 into wowdev:master Oct 28, 2023
1 check passed
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.

3 participants