Lichess4545 / heltour

Chess tournament management software for the Lichess4545 league
https://www.lichess4545.com/
MIT License
46 stars 38 forks source link

Time Zone info sometimes missing from pairing DM #410

Closed mathgeek closed 2 years ago

mathgeek commented 2 years ago

Originally submitted by @Unclevinny in Lichess4545/Chesster#376

Examples from recent chesster DMs:

This example was for two non-alts:

You have been paired for Round 4 in Season 27.
@akhmadzaki (white pieces) vs @unclevinny (black pieces)
Message your opponent here as soon as possible.
When you have agreed on a time, post it in #team-scheduling.

This pairing was for an alt (futyporyt) and a non-alt:

You have been paired for Round 4 in Season 27.
@osrswoodcutting (white pieces) vs @futyporyt(black pieces)
Message your opponent here as soon as possible.
When you have agreed on a time, post it in #team-scheduling.

This is a sporadic problem, and doesn't seem to be related to whether either user has their Slack time zone set, or a country chosen at lichess.

mathgeek commented 2 years ago

@Unclevinny I have a PR open for this but as far as I can tell it only happens for alternate notifications, if you see it again (after this fix is merged) let me know and I can dig into it more