console icon indicating copy to clipboard operation
console copied to clipboard

🚀 Improvement: Use Local Time for Premium Support Timings

Open ItzNotABug opened this issue 11 months ago • 5 comments

🔖 Feature description

Currently the timings for premium support are listed in UTC time format which may not be easy to convert to local time. Using a user's local standard time would be much easier to understand the exact timings for the support.


Current -


Suggestion -

🎤 Pitch

It would make it easier to understand support timings.

👀 Have you spent some time to check if this issue has been raised before?

  • [X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

ItzNotABug avatar Mar 06 '24 16:03 ItzNotABug

@ItzNotABug. thanks for creating this issue! 🙏🏼 Let me bring it up to the team.

If we move forward with this, would you like to work on making the change?

stnguyen90 avatar Mar 06 '24 17:03 stnguyen90

Go the green light for this so I'm going to open this up for contributors.

stnguyen90 avatar Mar 06 '24 17:03 stnguyen90

@ItzNotABug. thanks for creating this issue! 🙏🏼 Let me bring it up to the team.

If we move forward with this, would you like to work on making the change?

@stnguyen90 I could definitely give this a try.

I see these two pages here -

  1. https://github.com/appwrite/console/blob/main/src/routes/console/supportWizard.svelte
  2. https://github.com/appwrite/console/blob/main/src/lib/components/support.svelte

I assume we'd use Date() based logic but should that go in the same page's <script> tag or somewhere else?

ItzNotABug avatar Mar 06 '24 17:03 ItzNotABug

@ItzNotABug, yep! This is a little different since it's a day/time range, but maybe some of the following will help you in formatting it right:

https://github.com/appwrite/console/blob/0700c5cc45c1093a96aa279dfea815a64ac549f5/src/routes/console/project-%5Bproject%5D/messaging/wizard/step3.svelte#L33-L41

We have various helpers here too.

stnguyen90 avatar Mar 06 '24 22:03 stnguyen90

Hey @stnguyen90, I've attempted to raise a PR, however I wasn't sure of adding a test for the used date methods as I couldn't think of testing the expected local time against a set of expected results. Let me know how to proceed with the PR.

Thanks.

ItzNotABug avatar Mar 07 '24 10:03 ItzNotABug