Suggestions to improve AccountProperties

[dgarcia360]

Feedback after using the feature AccountPropertyTransaction:

1. AccountPropertyTransaction.createAddressFilter could be created with a different class (e.g. "FilterModification" or just "Filter"). The reason behind is to follow the same approach already implemented in MultisigCosignatoryModification when sending a multisig transaction.

Current approach

const filter = AccountPropertyTransaction
    .createAddressFilter(
        PropertyModificationType.Add,
        Address.createFromRawAddress('SC5DT5Y4DZ7ATYFSWSUX76FAWL6O4ZS2UZ5QSCWF')
    );

Proposed approach

const filter = new FilterModification(
    PropertyModificationType.Add, 
    Address.createFromRawAddress('SC5DT5Y4DZ7ATYFSWSUX76FAWL6O4ZS2UZ5QSCWF')
);

or

const filter = FilterModification.createForAddress(
    PropertyModificationType.Add, 
    Address.createFromRawAddress('SC5DT5Y4DZ7ATYFSWSUX76FAWL6O4ZS2UZ5QSCWF')
);

2. createEntityTypeFilter(modificationType: PropertyModificationType, entityType: number) accepts a number as an entityType parameter. To reduce typing errors, it would be better to limit the parameter to the TransactionType enumeration.

[rg911]

Fixed. should support both constructor / static method creation.