Page MenuHomePhabricator

PropertyDiff should include the data type
Closed, DeclinedPublic

Description

The property's data type should be considered when generating a diff. Even though we currently don't allow editing of the data type, it is not guaranteed that it will never change.

Details

Reference
bz41107

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 12:54 AM
bzimport set Reference to bz41107.
bzimport added a subscriber: Unknown Object (MLST).

We said we didn't want to allow changing the data type ever last we talked about this. I assume we can close this then.

(In reply to comment #1)

We said we didn't want to allow changing the data type ever

-> putting this as low & enh. Might be WONTFIX. Daniel?

It's valid but low prio. Remember that it's possible to view diffs between different pages (i.e. also between different properties). In that case, I'd want to see the data type in the diff.

Lydia_Pintscher removed a subscriber: Unknown Object (MLST).
Lydia_Pintscher removed a subscriber: Unknown Object (MLST).

With T95686 approaching, this should be higher.