Update needed to help with [ITS-223308] Send email field not switching to "no" after email is sent. This just required the addition of a minor method in Asset.py to call update and edit in one go, which will hopefully avoid the type errors the News view and controller in Tinker are running into.
This update also includes the addition of docstrings to many (but not all) of the more involved methods, as there were none before. Hopefully this will make the API easier to work with going forward! I also made some changes to the layout of config.dist.py, to make it more compatible with test.py and the Tinker config.py file in the future.
Size and Type of change
Small Change - 1 person needs to review this
New Feature (?)
How Has This Been Tested?
I ran the method from test.py and it behaved as desired with a few different key-value updates I tried on the test page.
Checklist:
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[ ] I have tested on multiple browsers (Chrome, Firefox, Safari, IE suite)
Description
Update needed to help with [ITS-223308] Send email field not switching to "no" after email is sent. This just required the addition of a minor method in Asset.py to call update and edit in one go, which will hopefully avoid the type errors the News view and controller in Tinker are running into.
This update also includes the addition of docstrings to many (but not all) of the more involved methods, as there were none before. Hopefully this will make the API easier to work with going forward! I also made some changes to the layout of config.dist.py, to make it more compatible with test.py and the Tinker config.py file in the future.
Size and Type of change
How Has This Been Tested?
I ran the method from test.py and it behaved as desired with a few different key-value updates I tried on the test page.
Checklist: