Skip to content

Add Documentation#2

Open
Droid00000 wants to merge 42 commits into
topi314:masterfrom
Droid00000:documentation
Open

Add Documentation#2
Droid00000 wants to merge 42 commits into
topi314:masterfrom
Droid00000:documentation

Conversation

@Droid00000
Copy link
Copy Markdown

Summary

So, this is my attempt at documentation. I mainly attempted to reference the docs for LavaSrc and the SponsorBlock plugin. If there's anything that's incorrect, please feel free to let me know. I'll attempt to correct it.

Copy link
Copy Markdown
Owner

@topi314 topi314 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not bad for a start, but I prefer to have tables with field descriptions, feel free to link to https://lavalink.dev for common types

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment on lines +44 to +58
Response:

```json5
{
"type": "normal",
"tracks": [
{
"encoded": "...",
"info": "{}",
"pluginInfo": "{}",
"userData": "{}"
}
]
}
```
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

responses should have a table and then a <details> with a json example

@Droid00000
Copy link
Copy Markdown
Author

Droid00000 commented Feb 3, 2025

Alright, I've added tables and descriptors for types and their custom objects, as well as wrapped the JSON examples in <details>. I wasn't sure if I needed to add examples for every response since most of them are just standard Lavalink objects. But, I can do that as well.

@Droid00000 Droid00000 requested a review from topi314 February 3, 2025 02:13
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md
Comment on lines +114 to +127
<details>
<summary>Example Payload</summary>

```json5
{
"type": "normal",
"tracks": [
{
"encoded":"QAAAjQIAJVJpY2sgQXN0bGV5IC0gTmV2ZXIgR29ubmEgR2l2ZSBZb3UgVXAADlJpY2tBc3RsZXlWRVZPAAAAAAADPCAAC2RRd"
}
]
}
```
</details>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no table?

Comment thread README.md Outdated
@Droid00000
Copy link
Copy Markdown
Author

Alright, I've tried to make the changes you've asked.

@Droid00000 Droid00000 requested a review from topi314 February 3, 2025 21:26
@topi314
Copy link
Copy Markdown
Owner

topi314 commented Feb 3, 2025

if you solved something feel free to mark it as resolved

Copy link
Copy Markdown
Owner

@topi314 topi314 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe the following structure would fit better:

  • Installation
  • API
    • Common Types
    • Endpoints
      • ...
    • Events
      • ...

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment on lines +94 to +102
```json5
{
[
{
"encoded": "QAAAjQIAJVJpY2sgQXN0bGV5IC0gTmV2ZXIgR29ubmEgR2l2ZSBZb3UgVXAADlJpY2tBc3RsZXlWRVZPAAAAAAADPCAAC2RRd"
}
]
}
```
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this payload doesn't look correct to me

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean, I couldn't figure out how else it wanted the payload. All that worked for me was taking an array of encoded tracks and then converting it directly to JSON

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you showed an object which contains an array, that's not valid json
it shiuld just be an array of objects

Comment thread README.md Outdated
@Droid00000
Copy link
Copy Markdown
Author

Update: 👋 I’m still around to work on this. Ideally, I’ll be able to apply your feedback sometime next week. Apologies for the delay

Comment thread README.md Outdated
Comment on lines +94 to +102
```json5
{
[
{
"encoded": "QAAAjQIAJVJpY2sgQXN0bGV5IC0gTmV2ZXIgR29ubmEgR2l2ZSBZb3UgVXAADlJpY2tBc3RsZXlWRVZPAAAAAAADPCAAC2RRd"
}
]
}
```
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you showed an object which contains an array, that's not valid json
it shiuld just be an array of objects

Comment thread README.md Outdated
Comment thread README.md Outdated
@Droid00000 Droid00000 requested a review from topi314 March 31, 2025 02:35
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md
@topi314
Copy link
Copy Markdown
Owner

topi314 commented Apr 4, 2025

just as a note, if you resolve something please also resolve the conversation

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.

2 participants