Codeinwp / Codeinwp/visualizer
Using , instead of . for decimal breaks the graphs
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 108
- Forks
- 29
- Ø Merge
- 1 T. 9 Std.
- Gemergte PRs (30 T.)
- 7
Beschreibung
In the data, if you use ',' for decimals, the graphs will not work properly.
Some countries use ',' as a decimal point instead of '.', for that reason, we should try to accommodate that use case.
It might just be a matter of replacing them:
str_replace ( '10,45' , '.' , ',');
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
Beginne damit, nachzuverfolgen, wie Chart-Daten geparst werden, bevor sie die Diagramme erreichen, und verwende dabei den im Issue beschriebenen Fall mit Dezimalkomma als Reproduktion. Identifiziere den relevanten Einstiegspunkt für die Eingabeverarbeitung sowie alle vorhandenen Prüfungen oder Tests; abgeschlossen ist die Aufgabe, wenn Werte wie 10,45 akzeptiert und korrekt dargestellt werden, ohne die Eingabe mit Dezimalpunkt zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- data-visualization
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 28/100