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

feat(1493): events have timezones #1800

Merged
merged 3 commits into from
Jan 6, 2024
Merged

feat(1493): events have timezones #1800

merged 3 commits into from
Jan 6, 2024

Conversation

jhackett1
Copy link
Contributor

fixes: #1493

db/schema.rb Outdated Show resolved Hide resolved
@matyikriszta matyikriszta force-pushed the timezone-events branch 2 times, most recently from 48c8f43 to 63b253e Compare October 25, 2022 01:22
Copy link
Contributor

@KimberleyCook KimberleyCook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@github-actions
Copy link

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the Stale label Jan 24, 2023
@biggianteye
Copy link
Contributor

This PR has been approved but hasn't been merged. Is there anything that's missing that would stop it from being merged (other than being out of sync given how old it is)?

@matyikriszta
Copy link
Contributor

@biggianteye yes, we have tested this PR further and it does not actually work. I'm going to remove the Approval for now. We'll have to revisit this issue, it's on our roadmap.

@matyikriszta matyikriszta marked this pull request as draft October 23, 2023 18:57
@matyikriszta matyikriszta force-pushed the timezone-events branch 5 times, most recently from 9467e31 to 5743d31 Compare January 4, 2024 03:09
@matyikriszta matyikriszta marked this pull request as ready for review January 4, 2024 03:24
db/schema.rb Outdated Show resolved Hide resolved
Copy link
Contributor

@gnclmorais gnclmorais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice change! Seems to work locally:
image

@matyikriszta matyikriszta merged commit 5d2170f into master Jan 6, 2024
5 checks passed
@matyikriszta matyikriszta deleted the timezone-events branch January 6, 2024 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update timezone to an "Event"
5 participants