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

Implement logs/email support #5

Closed
LucasFares opened this issue Nov 10, 2016 · 0 comments
Closed

Implement logs/email support #5

LucasFares opened this issue Nov 10, 2016 · 0 comments
Assignees

Comments

@LucasFares
Copy link
Contributor

Implement a way to inform the user when things happen.

@LucasFares LucasFares self-assigned this Nov 10, 2016
@LucasFares LucasFares removed this from Completed in Things to do Sep 7, 2017
johndiiorio pushed a commit that referenced this issue Feb 23, 2018
huss pushed a commit that referenced this issue Apr 6, 2021
wootent pushed a commit to wootent/OED that referenced this issue Jun 29, 2022
wootent pushed a commit to wootent/OED that referenced this issue Jul 15, 2022
hyperupcall added a commit to fox-forks/OED that referenced this issue Feb 19, 2023
* Initial Work (wip)

* updates/fixes

- Fixes for routes
- Got multiple files to work
- Fix some parameters
- Updated tests

* meter tests working

* partially changed

* hopefully final meter test & bug fix

- This fixes a race condtion when error with multiple meters in a single
CSV file. Details in code comment.

* Update more tests working (7,9,10)

* More tests working (51 of 52)
-expected formatted to work
-expected meters changed to have correct name
-expected 12 time changed

* export data downloads multiple files

* adding units to csv files, currently grabbing unitId

* adding units to csv file, pushing for dylan

* Make tables be true markdown

They should be easier to read and work with now.

* test description; update README

* remove test 28

* @bryanFowles and @DylanHill54 commit with export

* start tests for DST

This is a work in progress that is being saved.

* Add DST tests

13 new tests where cross DST. Needed to add filling in standard
units/conversions and meter unit. Also fixed a few issues in
descriptions of tests. Note pipe115 is failing and seems to be
issue with pipeline so needs further investigation.

* fixed format

* fix pipeline bug where result modified

In one case (end only, shift into DST), the pipeline modified the
endTimestamp. Because moment puts this change in the original object,
it changed the result already put into the array. The software now
clones the Reading before adding to result to avoid this.
Did in every case just to be safe.

* replace manual tests with auto ones

This removes all the files associated with the manual tests now that the
automated tests do them. It renames the testing file to be the same as
the old one with limited tests for the pipeline. It also separates
the data files into its own directory to match other tests.

* add all new meter values for meter tests

* fix comment typo found during PR creation

Too lazy to start PR again from scratch :-)

* Bump qs and body-parser

Bumps [qs](https://github.com/ljharb/qs) to 6.5.3 and updates ancestor dependency [body-parser](https://github.com/expressjs/body-parser). These dependencies need to be updated together.


Updates `qs` from 6.5.2 to 6.5.3
- [Release notes](https://github.com/ljharb/qs/releases)
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](ljharb/qs@v6.5.2...v6.5.3)

Updates `body-parser` from 1.18.3 to 1.20.1
- [Release notes](https://github.com/expressjs/body-parser/releases)
- [Changelog](https://github.com/expressjs/body-parser/blob/master/HISTORY.md)
- [Commits](expressjs/body-parser@1.18.3...1.20.1)

---
updated-dependencies:
- dependency-name: qs
  dependency-type: indirect
- dependency-name: body-parser
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump cookiejar from 2.1.2 to 2.1.4

Bumps [cookiejar](https://github.com/bmeck/node-cookiejar) from 2.1.2 to 2.1.4.
- [Release notes](https://github.com/bmeck/node-cookiejar/releases)
- [Commits](https://github.com/bmeck/node-cookiejar/commits)

---
updated-dependencies:
- dependency-name: cookiejar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* convert most strings to single quotes

* update description of tests

* update expected files to include line in name

* quick edit

* Bump http-cache-semantics from 4.1.0 to 4.1.1

Bumps [http-cache-semantics](https://github.com/kornelski/http-cache-semantics) from 4.1.0 to 4.1.1.
- [Release notes](https://github.com/kornelski/http-cache-semantics/releases)
- [Commits](kornelski/http-cache-semantics@v4.1.0...v4.1.1)

---
updated-dependencies:
- dependency-name: http-cache-semantics
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix issue with previousEnd validation

* resolve easy edit comments

* auto-format files

* Made refreshReadings changes in data.js

* remove selecte meters/groups from props

* remove units from props

* remove " in raw CSV reading column

* remove showRawExport from props

* remove timeInterval from props

* remove defaultLanguage as not actually used

* add missing param

* remove file sizes from state

* use identifier to label meter, unit

* get rid of array in graph export

- remove undefined check as does not seem needed.
- Rename function to include graph so clearer and parallel raw.

* comments, remove undefined check

* update dates in file names

- The end date is now the end time of last reading rather than
start time.
- Code was shifting times because it used to do multiple
meters/groups at once but removed since now single files.

* fix up raw export

- Uses state as was recently changed for non-raw export.
- Asks once if okay to download and moved logic. Now uses window alert.
- Fixed lint for trailing ,
- User output string now says file(s) since may do more than one.
- Getting raw readings now only takes a single meter value.
- Don't have DB return meter id on raw so don't have to remove.
- Remove meter identifier from route since only one meter.

* lint & comment

* work on raw reading export

- Directly route DB result. Code was doing moment language setting on
the server but that did not work since the locale is on the client.
- Model does not do usual row map since no meter id and made DB use
exact identifiers desired.
- SQL changed to use desired identifiers.
- Route returns values directly rather than map and putting into new
array. Note that was also leaving the original names so it was sending
extra, unused data.
- Use short JSON identifiers so route uses less data.
- Don't route meter identifier.
- Get meter identifier from state for file name.
- Remove eslint exception since seems not needed now.

* update file size estimate & user msg

* remove unwanted files

A couple of local files leaked into the PR and this removes them.

* export now React hooks + more

- The export container is gone and all logic is now in the component.
- The logic to do y-axis labels for line and bar graphics was refactored
into a new file as functions so can use with export
and graphics.
- The previous conversions for graph data exports was removed
and now it does it much more directly.
- The unit in the CSV now is the y-axis label so gives more
information such as the rate. The file name still uses the
unit identifier since shorter and now special characters.
- Where file names are created was standardized.

* fix unit exported for raw

Raw export gives the values for the meter. Instead of giving the meter
unit the code gives the unit with a conversion to this since this is
the normal graphic unit. It does the special cases where the
conversion is not 1,0.

* can import CSV that export

- Removed locale aware formatting of CSV date/time so
moment can import so upload works.
- Change how deal with header on CSV import so fact that first row has
more columns does not cause an issue.

* line with non-hour rate now have correct values

* fix bug if some items don't have readings

* changes from PR OpenEnergyDashboard#839

Redo changes from PR that were related to chartlinks. These were mostly
commits by @Jumpman5 but @huss did some, esp. for rate links.

* fix & location, remove commented out code

* minor fix for const

* map chartlinks now work

This was an old bug.

* fix bug where not setting bar duration in UI

* fix bug with processing chartlinks many times

The code now only processes chartlinks once to avoid issues.

* fix bug if create chartlink from window using chartlink

Needed to clear the parameters so don't get repeats.

* remove debug print

* fix linting

* correct name for group with compare

Also change to remove capitalization on "Has" for compare translations.

* fix unset state in export

The logic for lines was made similar to bars that protected against
unset state. This also caused the use of chartlinks with a defined
server range to fail due to the exception.

* update message for export file size warning

* chore: Remove unused dependencies

* chore: Move packagejson dev deps to `devDependencies`

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: William Brown <greenflare27@gmail.com>
Co-authored-by: Steven Huss-Lederman <huss@beloit.edu>
Co-authored-by: bryanFowles <bfowles@csumb.edu>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: spearec <colespeare@gmail.com>
Co-authored-by: Mushmou <noahjacinto03@gmail.com>
huss pushed a commit that referenced this issue Nov 8, 2023
Pulling OED development changes to my development branch
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

No branches or pull requests

1 participant