Skip to content

Consider timezones when autoScheduling#2068

Merged
Eddasol merged 1 commit intoequinor:mainfrom
Eddasol:fix-timezone
Mar 12, 2025
Merged

Consider timezones when autoScheduling#2068
Eddasol merged 1 commit intoequinor:mainfrom
Eddasol:fix-timezone

Conversation

@Eddasol
Copy link
Copy Markdown
Contributor

@Eddasol Eddasol commented Mar 10, 2025

Ready for review checklist:

  • A self-review has been performed
  • All commits run individually
  • Temporary changes have been removed, like console.log, TODO, etc.
  • The PR has been tested locally
  • A test have been written
    • This change doesn't need a new test
  • Relevant issues are linked
  • Remaining work is documented in issues
    • There is no remaining work from this PR that require new issues
  • The changes does not introduce dead code as unused imports, functions etc.

@github-actions
Copy link
Copy Markdown

🔔 Changes in database folder detected 🔔
Do these changes require adding new migrations? 🤔 In that case follow these steps.
If you are uncertain, ask a database admin on the team 😄

@Eddasol Eddasol marked this pull request as ready for review March 11, 2025 07:26
@Eddasol Eddasol requested a review from mrica-equinor March 11, 2025 08:17
@Eddasol Eddasol self-assigned this Mar 11, 2025
@Eddasol Eddasol force-pushed the fix-timezone branch 2 times, most recently from 6c02b9d to 40c25d4 Compare March 12, 2025 07:50
Comment thread backend/api/Database/Models/AutoScheduleFrequency.cs
Copy link
Copy Markdown
Contributor

@mrica-equinor mrica-equinor left a comment

Choose a reason for hiding this comment

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

LGTM :)

Copy link
Copy Markdown
Contributor

@mrica-equinor mrica-equinor left a comment

Choose a reason for hiding this comment

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

Check comment

@Eddasol Eddasol merged commit 96441c3 into equinor:main Mar 12, 2025
13 checks passed
@Eddasol Eddasol deleted the fix-timezone branch March 12, 2025 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants