Add edit_site method #180
Merged
Add edit_site method #180
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added put_site_info function that allow users to update metadata of a given site. This function call edit_site from pvsite_datamodel
added put_site_info route so a user can update site informations added test_put_site_info to test the route added test_put_site_and_update to test that informations are actually updated.
|
Looks great, Thanks @Bvr4 ! |
zakwatts
approved these changes
Jul 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request
Description
This PR refers to the issue #178 : edit site
Creation of the route and put_site_info method in main.py, that call edit_site function from pvsite_datamodel.
How Has This Been Tested?
Creation of 2 test functions:
test_put_site_info in test_main.py, testing the route
test_put_site_and_update in test_sites.py, testing that the site metadata is actually updated.
Yes
Checklist: