Skip to content

DBC22-6209: strip the time portion of the datetime string before parsing#1297

Merged
ray-oxd merged 1 commit intomainfrom
bugfix/DBC22-6209
Apr 16, 2026
Merged

DBC22-6209: strip the time portion of the datetime string before parsing#1297
ray-oxd merged 1 commit intomainfrom
bugfix/DBC22-6209

Conversation

@bcgov-brwang
Copy link
Copy Markdown
Collaborator

📝 Submitter

🔗 JIRA Ticket


✅ Quality Assurance & Requirements

  • Requirements Met: I have confirmed that all acceptance criteria from the JIRA ticket are fulfilled.
  • Tested desktop in local or dev envs
  • Tested mobile in local or dev envs
  • Ran unit tests locally
  • SonarCloud: I have verified that the SonarCloud analysis is clean/passing for this branch.

⚙️ Configuration & Environment

  • New Env Variables: Does this PR require new environment variables? (Yes/No)

    If yes, please list them here and ensure they are added to secret manager, the .env.example. and the Vault by an STA.

🧪 How to Test (if required)

  1. Verify if tasks pod log still report warnings like "[2026-04-14 09:37:26,886] WARNING:apps.event.tasks:Worker-1:unconverted data remains: T15:43:45-07:00
    "

🔍 Reviewer Checklist

  • Reviewed code for logic and cleanliness
  • Re-tested desktop/mobile in local or dev envs
  • Verified no new console warnings/errors
  • Confirmed that any new env variables are understood/documented

@bcgov-brwang bcgov-brwang requested a review from wmuldergov April 15, 2026 17:44
@bcgov-brwang bcgov-brwang marked this pull request as ready for review April 15, 2026 17:44
@ray-oxd ray-oxd force-pushed the bugfix/DBC22-6209 branch from cb237b5 to 085a7ec Compare April 16, 2026 03:36
@ray-oxd ray-oxd merged commit 8fa5e33 into main Apr 16, 2026
4 checks passed
@ray-oxd ray-oxd deleted the bugfix/DBC22-6209 branch April 16, 2026 03:38
@sonarqubecloud
Copy link
Copy Markdown

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