Godot-Mixing-Desk icon indicating copy to clipboard operation
Godot-Mixing-Desk copied to clipboard

API documentation

Open rezgi opened this issue 4 years ago • 3 comments

I have a bit of trouble finding the methods available to use the add-on like init_song() or play(song). It would be nice to have the list of all methods available and in what scopes ($MixingDeskMusic ? $Song ?...)

rezgi avatar Mar 09 '20 16:03 rezgi

Hi, I wrote some API docs for Mixing Desk Music in my fork, but I already renamed some of the funcs and did some cleanup. It might still help you: https://github.com/MJacred/Godot-Mixing-Desk/blob/cleanup/addons/mixing-desk/music/docs/api.md

I'll keep expanding/updating it, but my fork is very much still work-in-progress

Hope it helps, cheers!

MJacred avatar Mar 28 '21 15:03 MJacred

@MJacred that is one robust API doc. Would you mind if I used your doc as a basis for an API doc in my original repo?

kyzfrintin avatar May 04 '21 11:05 kyzfrintin

@kyzfrintin thank you! I was planning on doing a pull request once I got it all cleaned up (not sure I'll find time in May, but June should be fine).

The only thing missing should be applying song modifications before next loop run (I think you fixed that in this repo). And then rename some variables (not entirely happy yet).

Would you merge the fork if I made a pull request? I tried to adhere to your design. Although I would probably put all currently playing AudioStreamPlayers into the overlay, for consistency. Then the API doc could be added the same way and the code would have better readability.

MJacred avatar May 06 '21 22:05 MJacred