Codeinwp / Codeinwp/visualizer
Make data type row in CSVs optional
Ein zugehöriger Pull Request wurde bereits gemerged.
- #1059 von @vytisbulkevicius — gemerged
- Vorherrschende Sprache
- PHP
- Sterne
- 108
- Forks
- 29
- Ø Merge
- 1 T. 9 Std.
- Gemergte PRs (30 T.)
- 7
Beschreibung
Most data sets (e.g. https://www.quandl.com/api/v3/datasets/WIKI/AAPL.csv) do not provide the data type of the column as it is self-explanatory to humans. But because the plugin requires the data type to be specified, this prevents users from hooking directly into an API or a remote URL that does not provide the data type. We should make such integrations easier.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Im Issue werden keine Quelldateien oder Tests genannt. Prüfe zuerst die Issue-Details und den geschlossenen, gemergten Pull Request #1059. Verifiziere anschließend, ob CSV-Importe von URLs ohne eine Datentypzeile akzeptiert werden; abgeschlossen ist die Aufgabe, wenn solche Integrationen funktionieren, ohne diese Zeile zu erfordern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- backend
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 20/100