bot4dofus / Datafus

📚🥚 The Dofus database and socket events in JSON files. Includes the source code and more...
https://discord.gg/kNHsFcbUGp
MIT License
50 stars 13 forks source link

feat(U1): add TypeId<> and TypeIdVector<> #38

Closed ledouxm closed 11 months ago

ledouxm commented 1 year ago

This is a copy of this closed PR since i had some issues with the fork...

This is the original description for context:

Hi, i also developped a tool that does pretty much the same but i think having a single repo which we rely on could be a great thing for the ecosystem.

Sadly, as this issue states, the events.json file isn't complete enough to read a message.

This PR aims to fix this by adding two new types along with Vector :

I can clarify any point if needed !

LucBerge commented 1 year ago

No need to add the .json files in the PR

ledouxm commented 1 year ago

do you want the en and fr ones or i delete them too ?

LucBerge commented 1 year ago

Nothing in data/ must be manually commited. Checkout *.json please

ledouxm commented 1 year ago

So everytime you push something you have to delete the json files populated by the github action ?

LucBerge commented 1 year ago

So everytime you push something you have to delete the json files populated by the github action ?

No, its a mistake I've made... By delete, I mean checkout the *.json to the previous commit so that it does not appear in the PR. I tried to do it on the UI but with no success...

ledouxm commented 1 year ago

ok i've rollbacked them, it feels weird having json files not matching the ones generated by the code tho

ledouxm commented 11 months ago

do you plan on merging this PR ? if it doesnt fit your needs, tell me what to change

LucBerge commented 11 months ago

@ledouxm A tricky thing append while removing a branch: This issue has been automatically closed and I cannot reopen it. You will have to PR on master again...