PerformanC / NodeLink

Performant LavaLink alternative written with Node.Js
BSD 2-Clause "Simplified" License
38 stars 23 forks source link

Add poru to the supported clients #68

Closed Joniii11 closed 3 months ago

Joniii11 commented 3 months ago

Changes

I have added Poru to the supported clients list

Why

Because Poru now supports all of NodeLink's features and is still missing on this list.

Checkmarks

Additional information

None.

UnschooledGamer commented 3 months ago

Changes

I have added Poru to the supported clients list

Why

Because Poru now supports all of NodeLink's features and is still missing on this list.

Checkmarks

  • [x] The modified endpoints have been tested.
  • [x] Used the same indentation as the rest of the project.
  • [x] Still compatible with LavaLink clients.

Additional information

None.

Will be merged after @ThePedroo checks poru, Already asked for another review.

ThePedroo commented 3 months ago

Huh? It was missing?? 🤔

UnschooledGamer commented 3 months ago

Huh? It was missing?? 🤔

Missing what?

ThePedroo commented 3 months ago

Poru in the list, isn't it in LavaLink client list?

UnschooledGamer commented 3 months ago

Poru in the list, isn't it in LavaLink client list?

It's still in v3.7 list lol.

ThePedroo commented 3 months ago

This PR will be merged once https://github.com/parasop/poru/pull/107 is

UnschooledGamer commented 3 months ago

@ThePedroo, The main PR on poru's repo was merged I'll merge this too.

@Joniii11 Merging this PR, Marks Poru fully compatible with Both Versions of NodeLink.

Thanks.