materialsproject / api

New API client for the Materials Project
https://materialsproject.github.io/api/
Other
105 stars 34 forks source link

patch agreed_terms field for user settings #853

Closed yang-ruoxi closed 9 months ago

yang-ruoxi commented 9 months ago

Summary

Added terms agreement field to the patch method for user settings.
This will need to go out along with the front-end changes for checking our latest terms. By clicking the box on our website, it will patch an additional data field to our consumer store.

codecov-commenter commented 9 months ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (ffd9713) 70.39% compared to head (d2e8e94) 70.42%. Report is 5 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #853 +/- ## ========================================== + Coverage 70.39% 70.42% +0.02% ========================================== Files 51 51 Lines 2483 2482 -1 ========================================== Hits 1748 1748 + Misses 735 734 -1 ``` | [Files](https://app.codecov.io/gh/materialsproject/api/pull/853?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=materialsproject) | Coverage Δ | | |---|---|---| | [mp\_api/client/routes/\_user\_settings.py](https://app.codecov.io/gh/materialsproject/api/pull/853?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=materialsproject#diff-bXBfYXBpL2NsaWVudC9yb3V0ZXMvX3VzZXJfc2V0dGluZ3MucHk=) | `100.00% <ø> (ø)` | | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/materialsproject/api/pull/853/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=materialsproject)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.