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

6026: Improve timezones on org settings sent emails #6043

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

DariusPirvulescu
Copy link
Contributor

@DariusPirvulescu DariusPirvulescu commented Sep 19, 2024

What github issue is this PR for, if any?

Resolves #6026

What changed, and why?

I added the Users::Timezone.to_user_timezone helper which takes a time object/formatted string and it will adjust it to the current user's local timezone.

Opening it as Draft as it's work in progress. If you can please confirm that I'm on the right path.

How is this tested? (please write tests!) 💖💪

You can find a couple of tests under time_zone_spec.rb, just partial for now, need to add more related to handling odd arguments.
To run tests:

bundle exec rspec spec/controllers/concerns/users/time_zone_spec.rb:43

I am still new to testing, so please let me know if any changes required; I'm more than happy to adjust it.

Screenshots please :)

Run your local server and take a screenshot of your work! Try to include the URL of the page as well as the contents of the page.

Feelings gif (optional)

What gif best describes your feeling working on this issue? https://giphy.com/
How to embed:

![alt text](https://media.giphy.com/media/1nP7ThJFes5pgXKUNf/giphy.gif)

@github-actions github-actions bot added ruby Pull requests that update Ruby code Tests! 🎉💖👏 erb labels Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
erb ruby Pull requests that update Ruby code Tests! 🎉💖👏
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve timezones on sent emails page on org settings as admin
1 participant