Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add lip-remove_redundant_properties_in_transactions.md #11

Merged
merged 7 commits into from Apr 18, 2019

Conversation

@AndreasKendziorra
Copy link
Contributor

commented Apr 5, 2019

The current protocol allows for and partially enforces the usage of property-value pairs in transaction objects that are neither required nor used. These property-value pairs are contained in the JSON objects used to transmit transactions and in the input messages for the transaction signature and the transactionID generation. This increases the size of transactions and the complexity of the protocol unnecessarily. Therefore, this LIP proposes to remove these redundant properties of transaction objects.

@IkerAlus
Copy link
Member

left a comment

Everything looks good in the proposal.

@shuse2

shuse2 approved these changes Apr 10, 2019

@AndreasKendziorra AndreasKendziorra requested a review from karmacoma Apr 11, 2019

karmacoma added some commits Apr 18, 2019

@karmacoma karmacoma merged commit 91958bc into master Apr 18, 2019

@karmacoma karmacoma deleted the lip-remove_redudant_properties_from_transactions branch Apr 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.