But you should be able to test by changing the dev...

# confetti

[Marcello Galhardo]

But you should be able to test by changing the device date & hour to about 15 minutes before the session bookmarked.

[John O'Reilly]

cool, will try that

[Marcello Galhardo]

Remember to turn on the flag!

[John O'Reilly]

yeah, have that enabled

[John O'Reilly]

be great if you could do some sanity testing with version in play store too before I promote

[Marcello Galhardo]

Let me know how it goes.

[Marcello Galhardo]

I'm preparing dinner now but afterwards I can fix any bug you find. (:

[Marcello Galhardo]

Yes, also I think there was a merge conflict right? Would be nice to check if everything is working at runtime.

[Marcello Galhardo]

Notifications doesn't seem to be working for me in the internal version. What about you @John O'Reilly?

[John O'Reilly]

Only got chance to test just now and doesn't seem to be

[John O'Reilly]

does it work there locally with code from main?

[Marcello Galhardo]

It was working previously for me, yes. Not sure what happened.

[John O'Reilly]

was more wondering if maybe something changed after other PRs were merged....

[John O'Reilly]

could it be time zone issue I wonder

[Marcello Galhardo]

I'll have a look into that. 🤔

[yschimke]

This should probably be using the conference timezone, and adjusting.

[yschimke]

private fun createIntervalRange(): ClosedRange<LocalDateTime> {
        val now = dateService.now()
        val timeZone = TimeZone.currentSystemDefault()
        val future = (now.toInstant(timeZone) + INTERVAL).toLocalDateTime(timeZone)
        return now..future
    }

[John O'Reilly]

I had tried changing my time here to match time zone but still didn't seem to trigger notification

[John O'Reilly]

@Marcello Galhardo suspecting following was cause of issues I was seeing.....checking again now https://kotlinlang.slack.com/archives/C051P2HUVKP/p1680940894361469

[John O'Reilly]

I'm seeing SessionNotificationWorker startWorkRequest being invoked.....but not doWork for some reason

[Marcello Galhardo]

Nice, thank you for this info. Will help debug later!