Skip to content

Conversation

@mario99logic
Copy link
Contributor

@mario99logic mario99logic commented Nov 12, 2025

Description

Linear Ticket

UI-1940

What type of PR is this? (check all applicable)

  • 💡 (feat) - A new feature (non-breaking change which adds functionality)

Add timezone to trigger of type scheduler

@RonenMars
Copy link
Contributor

@mario99logic - make sure that the tests are passing and rebase this branch on top of main.

Copy link
Contributor

@RonenMars RonenMars left a comment

Choose a reason for hiding this comment

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

Generally, it's a good start in the frontend world, but it has critical issues that must be addressed.

Also, missing E2E tests:

  1. Create a scheduler trigger with timezone selection
  2. Edit scheduler trigger and change timezone
  3. Verify the timezone is saved and displayed correctly
  4. Test default UTC timezone behavior
  5. Test form validation with/without timezone

@mario99logic mario99logic force-pushed the mario/ui-1940-add_timezone branch from 736f4f3 to 5fd094f Compare November 20, 2025 10:11
@mario99logic mario99logic force-pushed the mario/ui-1940-add_timezone branch from f040135 to 85fc174 Compare November 20, 2025 13:24
@RonenMars
Copy link
Contributor

Please run rm -rf node_modules && npm i and commit the package-lock.json after the install run and the package-lock with the changes..

@mario99logic mario99logic force-pushed the mario/ui-1940-add_timezone branch from 68c3a5c to 5499f24 Compare November 25, 2025 06:51
@mario99logic mario99logic force-pushed the mario/ui-1940-add_timezone branch from 5499f24 to 6469626 Compare November 25, 2025 07:01
@mario99logic mario99logic merged commit 269d01d into main Nov 26, 2025
26 of 30 checks passed
@mario99logic mario99logic deleted the mario/ui-1940-add_timezone branch November 26, 2025 15:32
@github-actions
Copy link

🎉 This release is now available on GitHub

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants