getredash/redash

Google Spreadsheets worksheet by title cleanup/refactor

Open

#6.409 geöffnet am 31. Aug. 2023

Auf GitHub ansehen
 (5 Kommentare) (3 Reaktionen) (1 zugewiesene Person)Python (4.236 Forks)batch import
enhancementgood first issue

Repository-Metriken

Stars
 (24.845 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 9T 5h) (3 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfe die Kommentare und Codeänderungen in PR #5334, um die bestehende Implementierung zum Abrufen von Arbeitsblättern nach Titel zu verstehen. Überprüfe die Variablen mit gemischten Typen und entscheide dich für einen konsistenten Typ. Sieh dir an, wie Arbeitsblatttitel derzeit zitiert werden; erweitere die Unterstützung für nicht in Anführungszeichen gesetzte Einzelwort Titel. Überprüfe die 0/1 Indizierung von Arbeitsblattnummern, indem du die Google Sheets API Dokumentation und die aktuelle Verwendung im Codebase prüfst, und füge dann klärende Kommentare hinzu.
Tech Stack
python
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
GitPython
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60