luanti-org/luanti

Add a confirmation prompt to /clearobjects and /clearobjects quick

Aberta

#6.918 aberto em 15 de jan. de 2018

Ver no GitHubGuia do colaborador
 (8 comentários) (4 reações) (0 responsável)C++ (2.395 forks)auto 404
@ Server / Client / Env.Concept approvedLow priorityRequest / Suggestiongood first issue

Métricas do repositório

Stars
 (13.341 estrelas)
Métricas de merge de PR
 (Mesclagem média 13d 22h) (48 fundiu PRs em 30d)

Description

It already happened to me a lot of times that I accidentally typed /clearobjects instead of //clearobjects (command from World Edit to remove objects in a marked region) and had all objects on the server deleted.

I suggest to add a prompt (chat message) that asks the user if they really want to proceed with this action and that it might take a long time (in full mode).

If server owners run those commands by accident (for whatever reason), A LOT of progress could get lost depending on the game type and mods).

EDIT: Removed formspec as prompt option since a chat message is more than enough.

Guia do colaborador

Direção de pesquisa
Investigue a implementação atual de /clearobjects e /clearobjects quick no manipulador de comandos do servidor. Adicione uma etapa de confirmação que envia uma mensagem de chat ao usuário pedindo confirmação antes de executar a exclusão.
Pilha de tecnologia
lua
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
None
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65