Fix/properties forecast value

[peterdudfield]

Pull Request

Description

Make sure properties object is copied over from Forecast

How Has This Been Tested?

CI tests, and ran locally with API

• [ ] Yes

Checklist:

• [ ] My code follows OCF's coding style guidelines
• [ ] I have performed a self-review of my own code
• [ ] I have made corresponding changes to the documentation
• [ ] I have added tests that prove my fix is effective or that my feature works
• [ ] I have checked my code and corrected any misspellings

[codecov[bot]]

Codecov Report

Merging #245 (1b647e3) into main (3b753ca) will increase coverage by 0.01%. Report is 3 commits behind head on main. The diff coverage is 100.00%.

:exclamation: Current head 1b647e3 differs from pull request most recent head c141aaf. Consider uploading reports for the commit c141aaf to get more accurate results

@@            Coverage Diff             @@
##             main     #245      +/-   ##
==========================================
+ Coverage   95.25%   95.26%   +0.01%     
==========================================
  Files          27       27              
  Lines        1727     1731       +4     
==========================================
+ Hits         1645     1649       +4     
  Misses         82       82              

Files	Coverage Δ
nowcasting_datamodel/__init__.py	100.00% <100.00%> (ø)
nowcasting_datamodel/models/forecast.py	85.78% <100.00%> (+0.14%)	:arrow_up:
nowcasting_datamodel/read/read.py	98.07% <100.00%> (+0.01%)	:arrow_up:

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