getredash/redash

Google Spreadsheets worksheet by title cleanup/refactor

Open

#6409 aperta il 31 ago 2023

Vedi su GitHub
 (5 commenti) (3 reazioni) (1 assegnatario)Python (4236 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (24.845 star)
Metriche merge PR
 (Merge medio 9g 5h) (3 PR mergiate in 30 g)

Descrizione

In #5334 , I left a few comments about future work on the "fetch worksheet by title" functionality for the google spreadsheets query runner. We should address those, namely:

  • don't use mixed-type variables
  • allow for non-quoted worksheet titles (single word)
  • verify 0/1 indexing of worksheet numbers, and add comment if it differs

Guida contributor

Direzione di ricerca
Rivedi i commenti e le modifiche al codice nella PR #5334 per comprendere l'implementazione esistente per il recupero del foglio di lavoro per titolo. Controlla le variabili di tipo misto e decidi un tipo coerente. Osserva come i titoli dei fogli di lavoro sono attualmente quotati; estendi il supporto per titoli di una sola parola non quotati. Verifica l'indicizzazione 0/1 dei numeri dei fogli di lavoro esaminando la documentazione dell'API di Google Sheets e l'utilizzo corrente nella codebase, quindi aggiungi commenti chiarificatori.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60