Remove redundant specification on the Data Model

[dtrzesniak]

See that we use defaults as much as possible.
Example, we don't need to define a Display order if the default is the one we want

public record AmountType : KeyedOrderedDimensionWithExternalId, IHierarchicalDimension
{
[Display(Order = 10)] --> Not needed, over-specified
[Dimension(typeof(AmountType))]
public string Parent { get; init; }

[Display(Order = 20)]
[Dimension(typeof(PeriodType))]
public PeriodType PeriodType { get; init; }

}

[tkukuljan]

The default ordering is not sufficient only for Export purposes. Therefore I plan to keep the order attributes only for the records that are exported (that are ReinsurancePortfolio, InsurancePortfolio, GroupOfReinsuranceContract, GroupOfInsuranceContract, DataNodeState, InterDataNodeParameter, SingleDataNodeParameter)
Is this how it should be done?
@dcolleoni @dcolleoni @amuolo

[dtrzesniak]

@tkukuljan @amuolo @dcolleoni . In V7, the order for the UI and Export was taken from the order the properties appear in the data model (top to bottom). It could be that in SMC the order is alphabetical instead, and then indeed the Display order would be needed.
However, IMO if the behavior is not the natural data model order, then we should request this change to to the platform.

[tkukuljan]

@dtrzesniak So we leave the Display orders where needed and remove the rest?
Indeed the default in Export is alphabetical.

[dtrzesniak]

I would remove everywhere where not needed. So, yes, for now keep only those needed for Export.

[amuolo]

Closed since the PR #114 is merged.