-
Notifications
You must be signed in to change notification settings - Fork 39
Community api #427
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
base: master
Are you sure you want to change the base?
Community api #427
Conversation
|
This is indeed a good way to go forward. Thoughts, @SMUnlimited ? |
|
If your playing with commands we don't need to distinguish this command specifically different from the others as it is already a form of API and the others that already exist may also be of use for scripters. Where we put this doc, probably where we need to put in the Dev Manual documentation (Manual folder), where we could describe every command operation. That is still outstanding to be converted to wiki format though #215 |
|
As I said, these APIs are theoretically not meant to be used and need to be specifically distinguished, because AMAI Dev there is no need to disable own functionality. Need to consider specifically collecting interfaces that are only used by map makers If the existing numbering is continued for a long time, it may not be possible to distinguish them at a glance, just like the commands between 50-60. Without looking at the code, you have no idea what they are for, but if they are specifically divided into 5000, it is definitely for community use -- They can be easily distinguished from the coding rules. . This is my idea |
|
Regarding API namespacing (or number-spacing), there are 3 different kinds of commands to distinguish:
Regarding my tavern availability command, I was previously using cmd However, I wanted to standardize it as BTW, I didn't know that AI command data could be as high as 8000. I thought the limit was 255? I guess commands might be 32-bits, then... |
|
What I'm considering is that the quantity that can be accommodated between 80 and 90 is too small |
@Slayer95 I think this is what you need
I really can't handle the form, sorry