Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Data download button attempts to download HTM rather than CSV file #713

Closed
HeatherBrooks opened this issue Mar 7, 2024 · 2 comments
Closed

Data download button attempts to download HTM rather than CSV file #713

HeatherBrooks opened this issue Mar 7, 2024 · 2 comments
Assignees
@ptomasula
Labels
bug
Milestone
v0.17.1 Hotfixes ...

Comments

@HeatherBrooks
Copy link
Collaborator

Issue reported this morning through the WikiWatershed.org contact form. I tried downloading the data for a different site (OCTPA010) when logged in to my StroudCenter account and got the same results.

The data download for my site (jrm001) does not seem to be working. When I click the button, it immediately starts to download an HTML file rather than compiling the .CSV that would hold the data.

download-htm
@HeatherBrooks
Copy link
Collaborator Author

Jake Lemon from TU also reported this on the EnviroDIY Monitor My Watershed forum: https://www.envirodiy.org/topic/mmw-site-download-sensor-data/

@ptomasula ptomasula self-assigned this Mar 19, 2024
@ptomasula ptomasula added the bug label Mar 19, 2024
ptomasula added a commit that referenced this issue Mar 19, 2024
@ptomasula
Restore CSV Download
eaeeada 
Related Issue #713
The CSV download view was still pulling affiliation information based on a site registrations account. With the change to organizational ownership, a SiteRegistration's affiliation is not tied to a single account but multiple accounts, and the relationship is no longer 1-to-1 (i.e. site's are affiliated with multiple accounts). For those reasons I removed the affiliation related portions of the metadata causing the error but to a bad (stale) query strucutre.
@ptomasula
Copy link
Member

This is resolved and deployed to production.

@aufdenkampe aufdenkampe added this to the v0.17.1 bugfixes milestone Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ptomasula ptomasula

Labels
bug
Projects
None yet
Milestone
v0.17.1 Hotfixes and bug fixes
Development

No branches or pull requests
3 participants
@aufdenkampe @HeatherBrooks @ptomasula