fix: Remove "time" field from dataset if no time column exists

[osundwajeff]

#428

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
16347	15401	94%	0%	🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: bb0fdd0 by action🐍

[codecov]

Codecov Report

Attention: Patch coverage is 92.30769% with 2 lines in your changes missing coverage. Please review.

Project coverage is 94.09%. Comparing base (dad4f7e) to head (bb0fdd0).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
django_project/gap/providers/observation.py	71.42%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #429      +/-   ##
==========================================
+ Coverage   94.04%   94.09%   +0.04%     
==========================================
  Files         259      259              
  Lines       15841    15865      +24     
==========================================
+ Hits        14898    14928      +30     
+ Misses        943      937       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[danangmassandy]

@osundwajeff the to_json code is in ObservationParquetReaderValue and it's not calling to_json in parent class. So your change is not called by ObservationParquetReaderValue. Have you tested your changes? you can use arable parquet data in staging.

tomorrownow_gap/django_project/gap/providers/observation.py

Line 580 in 49a88dc

def to_json(self):

[danangmassandy]

LGTM thanks @osundwajeff