Date's timezones

[obrunsmann]

Hello,

we're migrating to this package from our own Stream API implementation. While all the timestamps coming from the API are UTC we also need to parse them as UTC to let them automatically convert into local time.

What I've done was:

DateTime.parse('${json['time'] as String}Z')

This worked perfectly fine and got converted in the local timezone.

Right now:

print(activity.time!.isUtc);

Prints false which is not correct - the time actually is UTC :/

[sachaarbonel]

Hey @obrunsmann, thanks for opening an issue. Fair point! I think bringing intl depedency in our SDK will solve your issue

final date = DateFormat("yyyy-MM-ddTHH:mm:ssZ").parse('2021-05-26T14:23:33.918391', true);
expect(date.isUtc, true);

[obrunsmann]

Yeah nice! So do you need something from me? I am here to test any time

[sachaarbonel]

@obrunsmann I'll push a fix this week and ping you when it's done so you can try out

[sachaarbonel]

hey, @obrunsmann I addressed this issue in this PR #167, along with the recent issues you opened (#165,#163), can you let me know if this works for you? Keep in mind that it is based on our recent release (0.4.0) that includes some breaking changes like supporting generics. Here is the release note in case you need it.

You can try it out like this:

dependencies:   
  stream_feed:
     git:
       url: https://github.com/GetStream/stream-feed-flutter.git
       ref: support-extraData-null-value
       path: ./packages/stream_feed

[obrunsmann]

Thank you for that fix! My first checks where not successful but I expect this issue on our side - I need to check the timezone integration.

I can give you feedback on Thursday.

[obrunsmann]

@sachaarbonel sorry for my late feedback. I upgraded to the latest version but still have the same issue. The timezone of the activity.time property is set to CET, which imo should be set to UTC, so that converting back .toLocal() then works 🤔

Can you confirm that?

Edit: found it, working on the fix 🤓