openclimatefix / pv-site-api

Site specific API for PV forecasting
5 stars 8 forks source link

remove clientSQL, get user sites, add checks if the user has access t… #99

Closed peterdudfield closed 10 months ago

peterdudfield commented 10 months ago

Pull Request

Description

How Has This Been Tested?

Checklist:

codecov[bot] commented 10 months ago

Codecov Report

Merging #99 (8d56f6c) into main (a260dcd) will increase coverage by 0.25%. Report is 1 commits behind head on main. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #99      +/-   ##
==========================================
+ Coverage   93.22%   93.48%   +0.25%     
==========================================
  Files          10       10              
  Lines         428      445      +17     
==========================================
+ Hits          399      416      +17     
  Misses         29       29              
Files Changed Coverage Δ
pv_site_api/__init__.py 100.00% <100.00%> (ø)
pv_site_api/_db_helpers.py 100.00% <100.00%> (ø)
pv_site_api/main.py 93.43% <100.00%> (+0.27%) :arrow_up:
pv_site_api/pydantic_models.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

peterdudfield commented 10 months ago

@braddf @devsjc @rachel-labri-tipton I',m gona merge this, but I welcome any comments that I can add afterwards