Project

General

Profile

Actions

Architecture #2167

closed

Enforce the event type value serialize

Added by Nicolas CHARLES almost 13 years ago. Updated almost 13 years ago.

Status:
Released
Priority:
1 (highest)
Category:
Architecture - Code maintenance
Target version:
Effort required:
Name check:
Fix check:
Regression:

Description

The current code for eventlog has a String for the eventtype, which can be quite error prone and more difficult to maintain

Creating type based event type would prevent typo as the compiler would watch any typo

Actions #1

Updated by Nicolas CHARLES almost 13 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100
Actions #2

Updated by François ARMAND almost 13 years ago

  • Status changed from Pending technical review to 10

Technical review seems OK.

Actions #3

Updated by Jonathan CLARKE almost 13 years ago

  • Status changed from 10 to Released

Closing these "Architecture" issues without a functional review - this step of the workflow has no sense here, there is no functional impact on code architecture changes (or there shouldn't be anyway!).

I think we should remove this step from the workflow for Architecture issues.

Actions

Also available in: Atom PDF