Changed “value” to “volume” in OHLCV (#27).

highcharts · Aug 30, 2024 · de782d1 · de782d1
1 parent 3202f0c
commit de782d1
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/docs/connectors/morningstar/time-series/ohlcv.md b/docs/connectors/morningstar/time-series/ohlcv.md
@@ -2,7 +2,7 @@
 
  This type yields OHLCV time series data for a single or multiple securities.
 
- Returns Open, High, Low, Close, Value for the securities specified.
+ Returns Open, High, Low, Close, Volume for the securities specified.
 
  > **NOTE:** At the moment only a single security is supported.
 

diff --git a/src/TimeSeries/Converters/OHLCVSeriesConverter.ts b/src/TimeSeries/Converters/OHLCVSeriesConverter.ts
@@ -38,7 +38,7 @@ import MorningstarURL from '../../Shared/MorningstarURL';
 interface OHLCV {
     Id: string;
     Date: number;
-    Value: [open: number, high: number, low: number, close: number, value: number];
+    Value: [open: number, high: number, low: number, close: number, volume: number];
 }
 
 
@@ -127,7 +127,7 @@ export class OHLCVSeriesConverter extends TimeSeriesConverter {
 
         table.deleteColumns();
 
-        const valueColumns = ['Open', 'High', 'Low', 'Close', 'Value'];
+        const valueColumns = ['Open', 'High', 'Low', 'Close', 'Volume'];
 
         table.setColumn('Date');
 

diff --git a/src/TimeSeries/TimeSeriesJSON.ts b/src/TimeSeries/TimeSeriesJSON.ts
@@ -89,7 +89,7 @@ namespace TimeSeriesJSON {
         high: number,
         low: number,
         close: number,
-        value: number
+        volume: number
     ];
 
 

diff --git a/test/unit-tests/TimeSeries/OHLCV.test.ts b/test/unit-tests/TimeSeries/OHLCV.test.ts
@@ -45,7 +45,7 @@ export async function ohlcvLoad (
             `${securityId}_High`, 
             `${securityId}_Low`, 
             `${securityId}_Close`, 
-            `${securityId}_Value`
+            `${securityId}_Volume`
         ],
         'Connector table should exist of expected columns.'
     );