getredash/redash

Google Spreadsheets worksheet by title cleanup/refactor

Open

#6 409 ouverte le 31 août 2023

Voir sur GitHub
 (5 commentaires) (3 réactions) (1 assigné)Python (24 845 stars) (4 236 forks)batch import
enhancementgood first issue

Description

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

Guide contributeur

Stack technique
python
Domaine
backendapi
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic Python knowledgeFamiliarity with Redash plugin structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
Review the comments and code changes in PR #5334 to understand the existing implementation for fetching worksheet by title. Check the mixed type variables and decide on a consistent type. Look at how worksheet titles are currently quoted; extend support for non quoted single word titles. Verify the 0/1 indexing of worksheet numbers by examining the Google Sheets API documentation and the current usage in the codebase, then add clarifying comments.