-
Notifications
You must be signed in to change notification settings - Fork 0
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
Pacific Power exclusion endpoint #315
Conversation
ChangesSee this PR in automated jobs to test the endpoint. The new endpoint adds new meters to the table with a I altered the database structure to have
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, most of my feedback was on the automated-jobs side. Tested the upload functionality with an early breakpoint.
upload-test-breakpoint-518.txt
As you can see by the RESPONSE...
logs at the bottom of attached log file above, 2 uploads were made to pacific_power_exclusion table, as expected (2 new meters).
Follow-up task (future PR): OSU-Sustainability-Office/automated-jobs#51 |
Added an endpoint to retrieve all Pacific Power Meters in the
pacific_power_exclusion
table of the database.See also:
#313
OSU-Sustainability-Office/automated-jobs#47
Testing
sam local start-api
from backend directory)TODO
exclude
field ofnew
new
inexclude
field