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.

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

Display server timezone in notifications in general. #2360

Closed
2 tasks done
TheBakaBandit opened this issue Nov 30, 2022 · 10 comments
Closed
2 tasks done

Display server timezone in notifications in general. #2360

TheBakaBandit opened this issue Nov 30, 2022 · 10 comments
Labels
area:notifications Everything related to notifications feature-request Request for new features to be added

Comments

@TheBakaBandit
Copy link

⚠️ Please verify that this bug has NOT been raised before.

  • I checked and didn't find similar issue

🛡️ Security Policy

Description

Timezone is still wrong in release in push notifications as reported in issue 40.

This is the notification:

image

This is the settings page:

image

👟 Reproduction steps

Set server and display timezone in settings, trigger a notification on a monitor.

👀 Expected behavior

The Time Zone and time in the notification would be as set in the settings.

😓 Actual Behavior

The Time Zone and time in the notification remain UTC

🐻 Uptime-Kuma Version

1.19.0-beta.0

💻 Operating System and Arch

Rocky Linux 8.6 (Green Obsidian)

🌐 Browser

Version 107.0.5304.108 (Official Build) (64-bit)

🐋 Docker Version

Docker version 20.10.2

🟩 NodeJS Version

No response

📝 Relevant log output

No response

@TheBakaBandit TheBakaBandit added the bug Something isn't working label Nov 30, 2022
@louislam louislam added feature-request Request for new features to be added and removed bug Something isn't working labels Dec 5, 2022
@louislam louislam changed the title Timezone in notifications Display server timezone in Discord notifications Dec 5, 2022
@migtarx
Copy link

migtarx commented Dec 9, 2022

I would say this issue is not just in discord notifications. It also happends in pushover as well.

@TheBakaBandit TheBakaBandit changed the title Display server timezone in Discord notifications Display server timezone in notifications in general. Dec 10, 2022
@TheBakaBandit
Copy link
Author

Yes, we have tested in mattermost too and the same thing occurs.

@befantasy
Copy link

same problem with slack

@EdyDeveloper
Copy link
Contributor

Does anyone know if this will be fixed?

@runchunzhang
Copy link

Does anyone know if this will be fixed?
image

@radiohead-lab
Copy link

Hi Everyone,
Issue is still there with Discord notifications! Any plan to fix it?
Thx in advance.

@Skyfay
Copy link

Skyfay commented Apr 18, 2023

I would also be happy if the time is displayed correctly in the Discord notifications. Is that so complicated to implement?

@jtxnn678
Copy link

Same Timezone issue.

Received notification Email contents:
[Nextcloud] [✅ Up] 200 - OK
Time (UTC): 2023-06-10 12:35:25.844

Actual local time is 20:35 (UTC +08:00)

@CommanderStorm
Copy link
Collaborator

CommanderStorm commented Jun 10, 2023

@TheBakaBandit I accidentally did not tag this issue in #3203

Could you close this issue?
A fix for this will be included in the next release (it is just around the corner)

@CommanderStorm CommanderStorm added the area:notifications Everything related to notifications label Dec 6, 2023
@CommanderStorm
Copy link
Collaborator

Closing as resolved in #3203

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:notifications Everything related to notifications feature-request Request for new features to be added
Projects
None yet
Development

No branches or pull requests

10 participants