Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GBFS API: fixed wrong timezone used #1389

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

hansmorb
Copy link
Contributor

@hansmorb hansmorb commented Oct 24, 2023

Ich komm hier leider nicht weiter, immer noch diese blöden Timezone issues.
closes #1388

@hansmorb hansmorb changed the title fixed DT-Object corresponding to wrong timezone #1388 GBFS API: fixed wrong timezone used Oct 24, 2023
@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

Merging #1389 (ab358a0) into master (a43aed3) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #1389   +/-   ##
=========================================
  Coverage     36.90%   36.90%           
  Complexity     2165     2165           
=========================================
  Files            83       83           
  Lines          8807     8807           
=========================================
  Hits           3250     3250           
  Misses         5557     5557           
Files Coverage Δ
src/API/GBFS/StationStatus.php 100.00% <100.00%> (ø)

@hansmorb hansmorb added bug Something isn't working question Further information is requested php Pull requests that update Php code labels Nov 6, 2023
@datengraben
Copy link
Contributor

Ich habe mir das vor paar Wochen angeschaut. Das was du gemacht hast war richtig, aber es gibt innerhalb der aufgerufenen Methoden(-Kette) noch Verwendung von timestamp mit lokaler Zeit. Meine das wäre innerhalb der Day und Week Klasse gewesen. Ich versuche den Patch nochmal auszugraben.

@futuretap
Copy link
Contributor

Wäre toll, wenn dieses Problem gefixt werden könnte. Denn es hält uns momentan davon ab, GBFS-Feeds von CB in unserer Wohin?-App einzubinden.

@hansmorb
Copy link
Contributor Author

@futuretap Leider fehlen uns dafür im Moment die Ressourcen. Wir bitten dich noch um Geduld. Falls du selber eine Idee für einen Patch hast gerne her damit.

@futuretap
Copy link
Contributor

Ich fürchte, ich stecke in dem Projekt zu wenig drin, um eine echte Hilfe bei diesem Problem zu sein.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working php Pull requests that update Php code question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GBFS API: Zeitzonen inkorrekt
3 participants