Skip to content
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

Add send command #1073

Merged
merged 4 commits into from
Oct 29, 2023
Merged

Add send command #1073

merged 4 commits into from
Oct 29, 2023

Conversation

Angelillo15
Copy link
Contributor

This pr adds a command to send players to others servers from console or in game with the permission velocity.command.send

@kyngs
Copy link
Contributor

kyngs commented Aug 28, 2023

I'm pretty sure that such a command is intentionally not included.

@carlosdiamon
Copy link

whaos

@Angelillo15
Copy link
Contributor Author

I'm pretty sure that such a command is intentionally not included.

I know Velocity isn't trying to be all-in-one, but I think it's a feature that is present in most servers that run under velocity.
Installing a plugin just for that feature seems a bit nonsensical to me, especially when it doesn't affect performance because in the end it's just a command that if you don't want to use it, just don't give permission and it would be like it wasn't there.
I fully understand that you do not want to merge it may seem stupid to you but take it more as a suggestion with the work done

@electronicboy
Copy link
Member

it's the general "do we want to maintain low effort 'it does a thing' commands in the proxy", especially when plugins exist and are generally much easier to work on than the proxy itself in terms of providing variance and fulfilling needs.

The lack of any command feedback on this at all is generally a non-starter to me, but, what would that even look like

@Nacioszeczek
Copy link
Member

afaik Bungee doesn't provide those commands in the proxy itself, instead there are plugins bundled with it that add them.
Would probably be better to - with permissions - bundle already existing plugins and not do redundant stuff.

@Dreeam-qwq
Copy link

use https://github.com/Elytrium/VelocityTools

@astei
Copy link
Contributor

astei commented Oct 29, 2023

I'm going to be the bad guy and say this is fine to include.

@astei astei merged commit 81145e4 into PaperMC:dev/3.0.0 Oct 29, 2023
1 check passed
astei pushed a commit that referenced this pull request Oct 29, 2023
skbeh pushed a commit to skbeh/Velocity that referenced this pull request Jan 17, 2024
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.

7 participants