Cronicle icon indicating copy to clipboard operation
Cronicle copied to clipboard

(Feature) Add default timezone in config file

Open xuanqing94 opened this issue 3 years ago • 5 comments

Summary

Currently, the timezone seems to be based on geolocation. This is not very convenient because I sometimes want to schedule in another timezone. Allowing users to set up their preferred timezone would be great.

image

Reference: in airflow, for example, there is default_timezone setting: https://airflow.apache.org/docs/apache-airflow/stable/timezone.html#default-time-zone.

xuanqing94 avatar Mar 10 '21 17:03 xuanqing94

And yes, the drop-down box gives the option to change the timezone, one common mistake I made is forgot to change it...

xuanqing94 avatar Mar 10 '21 17:03 xuanqing94

So does that dropdown gives you what you need? Default timezone just comes from system setting. In containers I just use TZ env variable to set desired timezone. Btw, are you using airflow? What's your opinion on it airflow vs cronicle?

mikeTWC1984 avatar Mar 12 '21 14:03 mikeTWC1984

So does that dropdown gives you what you need? Default timezone just comes from system setting. In containers I just use TZ env variable to set desired timezone.

The drop-down menu is inconvenient, the list is very long and (for some timezone) you need to scroll a long way to find the correct item.

I can't set the timezone for my working laptop. The interesting thing is the default timezone is not the tzinfo of the server but client...

Btw, are you using airflow? What's your opinion on it airflow vs cronicle?

I only played with airflow for a few days, I think the web interface it is overcomplex for my job, cronicle is done just right. But as the project grows, I don't know which is better.

xuanqing94 avatar Mar 12 '21 15:03 xuanqing94

OK. On UI it comes from the browser indeed. It's stored in app.tz property, so if you up to dirty tweaks you can even force it to has a specific value (e.g. you can add something likeif(app.username == 'admin') app.tz = 'America/Los_Angeles' somewhere in combo.js file) In general to add some user specific TZ for UI is pretty straightforward change.

mikeTWC1984 avatar Mar 16 '21 03:03 mikeTWC1984

OK. On UI it comes from the browser indeed. It's stored in app.tz property, so if you up to dirty tweaks you can even force it to has a specific value (e.g. you can add something likeif(app.username == 'admin') app.tz = 'America/Los_Angeles' somewhere in combo.js file) In general to add some user specific TZ for UI is pretty straightforward change.

Oh cool! Thanks for the nice idea!

xuanqing94 avatar Mar 16 '21 04:03 xuanqing94