Picolab/pico-engine

developer UI should not allow some picos to be deleted

Aberta

#495 aberto em 17 de dez. de 2020

Ver no GitHub
 (0 comentário) (0 reação) (0 responsável)TypeScript (10 forks)auto 404
1 - Readybugdocumentationgood first issue

Métricas do repositório

Stars
 (51 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Quoting from our documentation, "Currently, the UI does not allow you to delete a pico which has children (until you first delete all of its children, etc. recursively)."

We need to make this so.

While the engine can just up and delete an entire sub-tree of picos, we don't want this available from the UI. Such a thing should only be done programmatically, and is in the province of Wrangler.

What's asked for here is a simple UI change to not present the "delete" link for a pico that has any children.

Guia do colaborador

Direção de pesquisa
Identifique onde o botão de exclusão é renderizado no código da UI e adicione uma condição para verificar se o pico tem filhos antes de exibir o botão. Use a API existente ou o modelo de dados para determinar o status dos filhos.
Pilha de tecnologia
typescript
Domain
frontend
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.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
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.
90