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.

Sign up for GitHub

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

Jump to bottom

Проблем с теглене на валутите след смяната на часа (DST) #77

Closed
petarov opened this issue Apr 14, 2017 · 1 comment
Closed

Проблем с теглене на валутите след смяната на часа (DST) #77

petarov opened this issue Apr 14, 2017 · 1 comment
Labels
bug
Milestone
1.2

Comments

@petarov
Copy link
Member

petarov commented Apr 14, 2017

След смяната на часа на 26-ти Март, приложението праща следните заявки:

https://server/currencybg.server/api/currencies/today/2017-04-14T00:00:00.000+03:00

Сървърът обаче очаква следното:

https://server/currencybg.server/api/currencies/today/2017-04-14T00:00:00.000+02:00

@Tsvetoslav88 Проблемът май идва от DateTimeUtilsNew. Там explicit махаме 02:00, може би е по-добре да махаме зоната по име - Europe/Sofia? Трябва да се тества това.
@petarov petarov added the bug label Apr 14, 2017
@petarov petarov changed the title Проблем при обръщане на зоните след смяната на часа (DST) Проблем с теглене на валутите след смяната на часа (DST) Apr 14, 2017
@petarov petarov added this to the 1.2 milestone Apr 14, 2017
petarov added a commit that referenced this issue Apr 17, 2017
@petarov
Merge pull request #79 from vexelon-dot-net/bug/#77-TimeZone_Problem
6df75ba 
Üsed withZoneSameLocal(ZoneId instead of withZoneSameInstant
@petarov petarov mentioned this issue Apr 17, 2017
Merged
@petarov
Copy link
Member Author

petarov commented Apr 18, 2017

Всичко изглежда OK на staging-a.

@petarov petarov closed this as completed Apr 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.2
Development

No branches or pull requests
1 participant
@petarov