Skip to content

Allow for TZ as a setting #49

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

Open
gitcnd opened this issue Mar 25, 2025 · 2 comments
Open

Allow for TZ as a setting #49

gitcnd opened this issue Mar 25, 2025 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@gitcnd
Copy link

gitcnd commented Mar 25, 2025

first off - this is the best extension ever!!! - saves me over-and-over.

  • new specstories are using utc - probably not what most people expect.
  • upon a restart, previous utc-tagged spectories were all erased, and new copies of it using my localtime appeared... ok, but, this trashed all the actual file dates and times on all my old specstory files in-so-doing...
  • would have been better to use localtime in the firstplace, and not change/rename files (which polluted my git commit with a pile of deletes and new files)
@belucid belucid added the enhancement New feature or request label Apr 26, 2025
@belucid
Copy link

belucid commented Apr 26, 2025

first off - this is the best extension ever!!! - saves me over-and-over.

Thanks @gitcnd That's great to hear.

new specstories are using utc - probably not what most people expect.

We debated this long and hard internally (probably spending more time on it than we should) and ultimately decided on UTC as the best approach as we are seeing SpecStory histories getting checked into repositories a lot and are talking to teams sharing their AI interaction histories via SpecStory and they aren't all in the same TZ. We felt UTC optimized for a team setting.

I think ideally what we should do is default to UTC, but make the TZ a setting so if folks are working alone, or in the same TZ as others, they can use something that's more natural to them. Using this issue to capture this as an enhancement.

@belucid belucid changed the title localtime messups Allow for TZ as a setting Apr 26, 2025
@belucid belucid assigned bago2k4 and unassigned tenpaiyomi May 23, 2025
@belucid
Copy link

belucid commented May 26, 2025

This is planned for the 0.12.2 later this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants