apache/superset

supersetCanCSV property naming could be better

Chiusa

#24.290 aperta il 5 giu 2023

Vedi su GitHubGuida contributor
 (12 commenti) (1 reazione) (0 assegnatari)TypeScript (17.291 fork)batch import
good first issuerisk:breaking-change

Metriche repository

Star
 (72.837 stelle)
Metriche merge PR
 (Merge medio 31g 15h) (773 PR mergiate in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Cerca nel codebase tutte le occorrenze di 'supersetCanCSV' e sostituiscile con 'supersetCanDownload'. Assicurati di aggiornare tutti i riferimenti nei file JavaScript/TypeScript, nei test e nella documentazione. Conferma che la nuova denominazione sia coerente con i modelli esistenti.
Tech stack
typescriptreact
Dominio
frontenddeveloper experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Familiarity with JavaScript/TypeScriptBasic knowledge of Apache Superset codebase
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90