Remove unused fields from serialized state

maryvilledev / codesplainUI

Web app for breaking down and annotating Python 3 source code.

https://www.codesplain.io

GNU General Public License v3.0

2 stars 2 forks source link

Remove unused fields from serialized state #497

Closed smokinGun closed 7 years ago

smokinGun commented 7 years ago

Description

The "snippetKey" and "hasUnsavedChanged" fields will no longer be included in the serialized JSON object for snippets. They are stripped out of the state using an axios "transformRequest" function.

Motivation and Context

fixes #421

Checklist:

[x] My code follows the code style of this project.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Browsers

[ ] Chrome
[x] Firefox
[ ] Safari
[ ] Mobile:
[ ] Other:

thebho commented 7 years ago

Is this good to go?

smokinGun commented 7 years ago

Yes, as well as PR #499.

dane-johnson commented 7 years ago

We need another review on both of them