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.

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

[PR] Revamp sec scraper backend #269

Merged
merged 5 commits into from
Oct 20, 2023
Merged

Conversation

solderq35
Copy link
Contributor

Changes

Move meter lookup table into automated-jobs repo to simplify future maintenance.

Tests

Tested on

  • Local webscraper test (Run energy dashboard backend on http://localhost:3000, test upload with automated-jobs webscraper pointed at port 3000)
  • Local website test (Run energy dashboard frontend on http://localhost:8080 while energy dashboard backend is also running locally, check OSU Operations for a solar meters building example)
  • Checked that data was uploaded as expected in local test, in MySQL database

Misc

  • note to self: With current implementation, must keep meters 123, 124 with name "Solar_Meters" in SQL database for now

@solderq35
Copy link
Contributor Author

forgot to merge master into branch before making changes, so ignore the first 3 commits. Will be squashed anyways

@solderq35 solderq35 merged commit 7a09726 into master Oct 20, 2023
3 checks passed
@s-egge s-egge deleted the revamp-sec-scraper-backend branch July 1, 2024 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant