apache/superset

supersetCanCSV property naming could be better

Open

#24.290 aberto em 5 de jun. de 2023

Ver no GitHub
 (12 comments) (1 reaction) (0 assignees)TypeScript (72.837 stars) (17.291 forks)batch import
good first issuerisk:breaking-change

Description

Description

Javascript code uses supersetCanCSV property to check wether user is allowed to download chart as csv, excel or image.

This is bit outdated, supersetCanDownload would be more informative?

Guia do colaborador

Pilha de tecnologia
typescriptreact
Domain
frontenddeveloper experience
Tipo Issue
refactor
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.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Familiarity with JavaScript/TypeScriptBasic knowledge of Apache Superset codebase
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90
Direção de pesquisa
Search the codebase for all occurrences of 'supersetCanCSV' and replace them with 'supersetCanDownload'. Ensure that all references in JavaScript/TypeScript files, tests, and documentation are updated. Confirm that the new naming is consistent with existing patterns.