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

Make reports available to all #970

Merged
merged 66 commits into from
Sep 7, 2015
Merged

Make reports available to all #970

merged 66 commits into from
Sep 7, 2015

Conversation

MilosKozak
Copy link
Contributor

No description provided.

@MilosKozak
extracted needed files
257cb4a
@MilosKozak
added report files
5951797
@MilosKozak
make it at least load without errors
e94aa85
@MilosKozak
some renames
f3115f2
@MilosKozak
make reports plugins
df0b8dc
@MilosKozak
removed tabulators, fixed idents
a837f22
@MilosKozak
doublequotes to quotes
f61a772
@MilosKozak
=== and !==
e3cb1da
@MilosKozak
maybePreventDefault
1e16946
@MilosKozak
if brackets
643cf00
@MilosKozak
codacy pass 1
fd224a2
@MilosKozak
codacy pass 2
9146949
@MilosKozak
codacy pass 3
2fdb03b
@MilosKozak
make html plugin's code dymnamicaly injected on load
03e1652
@MilosKozak
codacy pass 4
4129a29
@MilosKozak
restore replaced new code in treatments.js
bd4b72e
@MilosKozak
rebase changes
56ae37c
@MilosKozak
allow regex in query
2a24bc4
@MilosKozak
.x -> .mills everywhere to make calibration.js work
11283be
@MilosKozak
units passed with options
b1cc6b1
@MilosKozak
plugin.html must be a function(client) to be correctly translated wit…
c78e9b0 
…h initialized language module
@MilosKozak
require not needed anymore
f8079d4
@MilosKozak
codacy pass 5
549933a
@MilosKozak
gui improvements
89f184a
@MilosKozak
make careportal events dynamic
e1c5977
@MilosKozak
careportal test fix
db784e5
@MilosKozak
careportal codacy fix
69c0121
@MilosKozak
reuse careportal's event list in reports
3ac1c71
@MilosKozak
fixed bug in processing request response
907f22d
@MilosKozak
resolved translation issue between eventType value and text in dropdo…
555fdca 
…wnbox
MilosKozak and others added 18 commits September 4, 2015 12:08
@MilosKozak
even more treatments.js refactoring
84259d9
@MilosKozak
Merge pull request #918 from MilosKozak/wip/reports
32e99c9 
Work on reports
@MilosKozak
fix mg/dl vs mg/dL issue
816caa2
@MilosKozak
Merge pull request #926 from MilosKozak/wip/reports
2f64c15 
fix mg/dl vs mg/dL issue
@MilosKozak
use profile timezone
68cf970
@MilosKozak
Merge pull request #928 from MilosKozak/wip/reports
c5b4b3d 
use profile timezone
@MilosKozak
1 day shift fix
bbe760a
@MilosKozak
Merge pull request #929 from MilosKozak/wip/reports
7dc6af6 
1 day shift fix
@MilosKozak
fixed day name too
2164ea2
@MilosKozak
Merge pull request #931 from MilosKozak/wip/reports
b9f69a8 
fixed day name too
@MilosKozak
another date issue
1ab11de
@MilosKozak
Merge pull request #939 from MilosKozak/wip/reports
29a6541 
another date issue
@MilosKozak
solved axis timezone issues
6f10f4d
@MilosKozak
removed unneeded code
47d3ab5
@MilosKozak
improved 12h axis format
8a1fbe9
@MilosKozak
Merge pull request #953 from MilosKozak/wip/reports
03529c5 
axis updates
@jasoncalabrese
merge in dev, update icon for report page
cd3f273
@jasoncalabrese
get thresholds from settings
de1f000
@codecov-io
Copy link

Current coverage is 72.28%

Merging #970 into dev will decrease coverage by -11.40% as of 4f073b4

@@              dev    #970   diff @@
=====================================
  Files          61      71    +10
  Stmts        4308    5135   +827
  Branches        0       0       
  Methods         0       0       
=====================================
+ Hit          3605    3712   +107
  Partial         0       0       
- Missed        703    1423   +720

Review entire Coverage Diff as of 4f073b4

Uncovered Suggestions

  1. +0.53% via ...lugins/dailystats.js#25...51
  2. +0.47% via ...ucosedistribution.js#38...61
  3. +0.47% via ...gins/calibrations.js#63...86
  4. See 7 more...

Powered by Codecov. Updated on successful CI builds.

jasoncalabrese added a commit that referenced this pull request Sep 7, 2015
@jasoncalabrese
Merge pull request #970 from nightscout/wip/reports
4d8c698 
Make reports available to all
@jasoncalabrese jasoncalabrese merged commit 4d8c698 into dev Sep 7, 2015
@jasoncalabrese jasoncalabrese deleted the wip/reports branch September 7, 2015 09:04
@khalavak khalavak mentioned this pull request Jul 7, 2023
Open
@quazarcd quazarcd mentioned this pull request Jul 7, 2023
Open
@raghumovva raghumovva mentioned this pull request Jul 7, 2023
Open
@dustin-lennon dustin-lennon mentioned this pull request Jul 7, 2023
Closed
@rguetschow rguetschow mentioned this pull request Jul 7, 2023
Open
@jweismann jweismann mentioned this pull request Jul 8, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MilosKozak @codecov-io @jasoncalabrese @scottleibrand