bugs found in functional testing on cluster

[scottleedavis]

### 1) display time is correct on local, but not on cluster . fixed by timezone fix

### 2) 'at' time fires local, but not on cluster . fixed by timezone fix

[scottleedavis]

/remind me check at <some time> is not triggering fixed. timezone issue

[tschlue]

Mattermost Version: 5.10.0
remind Version: 0.2.0

[scottleedavis]

@tschlue could you share the command that produced this output?

[tschlue]

/remind me in 1 min

[scottleedavis]

Ah! that should return an error instead of schedule a reminder. thank you.
/remind me a message in 1 min (e.g. /remind me|@user|~channel message time)

[tschlue]

Okay, but this doesn't fix the time problem.

[tschlue]

I think would like to use empty messages, just to remind me to look at mattermost or to wakeup a channel.

[scottleedavis]

For empty messages you can use empty double quotes. e.g. /remind me "" in 1 min. Slackbot doesn't permit empty messages except in this fashion as well.

I hadn't noticed the time issue until now. Could you share /remind __user ? What server tech are you using? docker? debian? other? Could you try Experimental -> timezone enabled?

[tschlue]

System
17:46(Only visible to you)
×
locale: en
location: America/Atikokan

[scottleedavis]

@tschlue could you try the bot-test plugin to see if it matches the header time of the post?
/test-time returns time. should match post header
https://github.com/scottleedavis/mattermost-plugin-bot-test/releases/tag/v0.1.2

[tschlue]

[scottleedavis]

@tschlue I could replicate this by changing my timezone on my laptop. I could also fix it by enabling experimental timezone support. Would you try that out and let me know?

System Console -> Experimental Features -> Timezone = true

[tschlue]

@scottleedavis This fix the problem apparently but is this necessary?

[scottleedavis]

@tschlue No, however it is quickest. I created #94 to take webapp timezone detection into account.

[tschlue]

thanks