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

[Request] make timezone for achievements more prominent #5007

Closed
jonyo opened this issue May 4, 2024 · 2 comments
Closed

[Request] make timezone for achievements more prominent #5007

jonyo opened this issue May 4, 2024 · 2 comments
Labels
bugfix release relevant Issue should be looked at for backporting into the next bugfix release done Done but not yet released request Feature request
Milestone

Comments

@jonyo
Copy link

jonyo commented May 4, 2024

Is your feature request related to a problem? Please describe.

When I first saw achievements, the time based ones were wrong. I got TGIF on a Thursday, "night owl" in the afternoon, etc.

There was no indication in the achievement that it was based on specific timezone. I found the timezone setting eventually, but for the first week or so I didn't look that close and just thought it was a new feature that was a little buggy. And I did not see any CTA to set the timezone (that I can remember).

Describe the solution you'd like

2 possible things:

  1. Call to action to set the timezone in the UI. Maybe as a notification or something.
  2. Ability to reset just the time/date based achievements - it bugged me I had a bunch for the wrong thing so I cleared the plugin data, and that cleared all my achievements not just time/date ones. So it would be cool to have some way in the UI to reset all time related. Orrr... Better yet, add a "reset this achievement" on individual ones to remove it if you want. (maybe that action needs the special permissions, making you enter password 😂 )

Describe alternatives you've considered

  • Another idea, expanding on the first one more: You could have a "setting the timezone" achievement! Name it something snazzy though. Maybe Keeping Time(zone) or something? ¯\_(ツ)_/¯
  • Maybe all time related achievements are "locked" until you specifically set the timezone and confirm it. And some sort of CTA to bring it to your attention.
  • If you don't lock them, at least show something in the UI saying Timezone: ___ in gray in the achievements list under each achievement that is affected, with link taking you to edit the plugin's settings... Maybe it would show on the achievement unlock notice as well, if I had seen that I would have probably fixed it right away.
  • Add a "use local timezone" button that would use JS to get the localized timezone (though I know from experience that can be tricky... maybe it is easier these days though I haven't tried in a while).

Additional context

No response

@github-actions github-actions bot added the request Feature request label May 4, 2024
@foosel
Copy link
Member

foosel commented May 7, 2024

I've done this:

  • timezone information, "time based" indicator and link to configuration in achievements overview
    image
  • ability to reset time based or all achievements from the settings
    image
  • offer to reset all time based achievements on change of the configured timezone
    image

Using the client side timezone wouldn't work - there might not be a client connected at the time the achievement is unlocked, so I have to rely on the server time. I tried squeezing a link to the config into time based achievement unlock notifications, but everything I tried to achieve that just looked horrible.

Will be in 1.10.1

@foosel foosel added done Done but not yet released bugfix release relevant Issue should be looked at for backporting into the next bugfix release labels May 7, 2024
@foosel foosel added this to the 1.10.x milestone May 7, 2024
@jonyo
Copy link
Author

jonyo commented May 7, 2024

The screenshots look great, I like that combination of solutions. The option to reset all from the settings page is good touch as well, not everyone would think to try clearing the plug-in data on a different part of the ui if they did want to reset all for whatever reason.

🎉 looking forward to the update!

@foosel foosel closed this as completed in af863e6 May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix release relevant Issue should be looked at for backporting into the next bugfix release done Done but not yet released request Feature request
Projects
Status: Done
Development

No branches or pull requests

2 participants