-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
enhancementNew feature or requestNew feature or requestin-progressWork currently in progressWork currently in progress
Description
Follow-up from #706 (closed by #708 which covered Phase 2).
Remaining ~27 occurrences
Batch 3: Services (~21 occurrences)
| File | Count | Notes |
|---|---|---|
| DataContext.cs | 2 | CreateTime/UpdateTime in SaveChanges |
| WTMLogger.cs | 2 | ActionLog timestamps |
| LoginUserInfo.cs | 1 | TimeTick property |
| AnalysisExcelExporter.cs | 1 | Export timestamp |
| WtmFileProvider.cs | 1 | UploadTime |
| WtmDataBaseFileHandler.cs | 1 | UploadTime |
| DashboardDefinition.cs | 2 | Property initializers (skip — no DI) |
| JsonFileDashboardService.cs | 2 | CreatedAt/UpdatedAt |
| EtlQuartzJob.cs | 6 | Job execution timestamps |
| EtlSchedulerService.cs | 1 | Schedule check |
| EtlPipelineExecutor.cs | 1 | Pipeline start time |
| QuartzHostService.cs | 1 | Delayed start |
Batch 4: DateRange static factories (~6 occurrences)
| File | Count | Notes |
|---|---|---|
| DateRange.cs | 6 | Add optional TimeProvider? parameter |
Also clean up
- FrameworkFilter.cs: 2 remaining
?? DateTime.Nowfallbacks from Phase 2
Skip (by design)
- DashboardDefinition property initializers (no DI at construction)
- .cshtml cache-bust, TypeExtension random data, codegen .txt templates
Key rule
- UTC contexts: use
.UtcDateTime, NOT.DateTime(preserves DateTimeKind.Utc)
Work: M | Phase: 3
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestin-progressWork currently in progressWork currently in progress