Skip to content

Commit

Permalink
Build changeS:
Browse files Browse the repository at this point in the history
- replace export default with export for classes
- change the entry point to the dist built file
- preserve the names of the classes and functions
  • Loading branch information
ClaudiaGivan committed Feb 21, 2025
1 parent 2b2bec0 commit 41af796
Show file tree
Hide file tree
Showing 21 changed files with 85 additions and 114 deletions.
72 changes: 24 additions & 48 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@
"name": "graphs-renderer",
"version": "1.0.0",
"description": "A library for rendering the graphs: CFD, Scatterplot and Histogram with d3.js",
"main": "./src/index.js",
"main": "./dist/graphs-renderer.js",
"module": "./dist/graphs-renderer.js",
"type": "module",
"scripts": {
"build:dev": "webpack --mode development",
Expand All @@ -24,6 +25,7 @@
"babel-loader": "^9.2.1",
"d3": "^7.9.0",
"imports-loader": "^4.0.1",
"terser-webpack-plugin": "^5.3.11",
"webpack": "^5.96.1",
"webpack-cli": "^5.1.4"
},
Expand Down
4 changes: 1 addition & 3 deletions src/graphs/ObservationLoggingService.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
class ObservationLoggingService {
export class ObservationLoggingService {
observationsByService = { data: { rows: [] } };
btoaToken = '';
serviceId = '';
Expand Down Expand Up @@ -68,5 +68,3 @@ class ObservationLoggingService {
return observation;
}
}

export default ObservationLoggingService;
2 changes: 1 addition & 1 deletion src/graphs/Renderer.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import * as d3 from 'd3';
/**
* Represents a generic graphs renderer
*/
export default class Renderer {
export class Renderer {
margin = { top: 30, right: 40, bottom: 70, left: 40 };
width = 1040 - this.margin.left - this.margin.right;
height = 460 - this.margin.top - this.margin.bottom;
Expand Down
4 changes: 2 additions & 2 deletions src/graphs/UIControlsRenderer.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
import { addDaysToDate, calculateDaysBetweenDates, readJsonFile } from '../utils/utils.js';
import Renderer from './Renderer.js';
import { Renderer } from './Renderer.js';
import * as d3 from 'd3';

/**
* Extends Renderer to manage UI controls the graphs
*/
export default class UIControlsRenderer extends Renderer {
export class UIControlsRenderer extends Renderer {
selectedTimeRange;
preventEventLoop;
chartName;
Expand Down
14 changes: 6 additions & 8 deletions src/graphs/cfd/CFDGraph.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import * as d3 from 'd3';
/**
* Class representing a Cumulative Flow Diagram (CFD) Graph Data
*/
class CFDGraph {
export class CFDGraph {
/**
* Creates a new CFDGraph instance.
* @constructor
Expand Down Expand Up @@ -83,7 +83,7 @@ class CFDGraph {
date: currentDate,
};
this.states.forEach((state, index) => {
dataEntry[state] = this.#getNoOfTicketsInState(this.states[index], currentTimestamp);
dataEntry[state] = this.getNoOfTicketsInState(this.states[index], currentTimestamp);
});
dataSet.push(dataEntry);
}
Expand All @@ -100,14 +100,14 @@ class CFDGraph {
* @param {number} timestamp - The timestamp at which to check the ticket state.
* @returns {number} noOfTickets - The count of tickets in the specified state for the given timestamp.
*/
#getNoOfTicketsInState(state, timestamp) {
getNoOfTicketsInState(state, timestamp) {
return this.data.filter((d) => {
if (!d[state]) {
return false;
}
let nextState = this.#getNextState(state);
let nextState = this.getNextState(state);
while (nextState !== null && !d[nextState]) {
nextState = this.#getNextState(nextState);
nextState = this.getNextState(nextState);
}
if (!d[nextState]) {
return d[state] <= timestamp;
Expand All @@ -123,10 +123,8 @@ class CFDGraph {
* @param {string} state - The current state of the ticket.
* @returns {string|null} nextState - The next state in the ticket lifecycle, or null if there is no next state.
*/
#getNextState(state) {
getNextState(state) {
const index = this.states.indexOf(state);
return index >= 0 && index < this.states.length - 1 ? this.states[index + 1] : null;
}
}

export default CFDGraph;
6 changes: 2 additions & 4 deletions src/graphs/cfd/CFDRenderer.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
import { calculateDaysBetweenDates, areDatesEqual, formatDateToLocalString } from '../../utils/utils.js';
import UIControlsRenderer from '../UIControlsRenderer.js';
import { UIControlsRenderer } from '../UIControlsRenderer.js';
import styles from '../tooltipStyles.module.css';
import * as d3 from 'd3';

/**
* Class representing a Cumulative Flow Diagram (CFD) graph renderer
*/
class CFDRenderer extends UIControlsRenderer {
export class CFDRenderer extends UIControlsRenderer {
#colorPalette = ['#22c55e', '#bbf7d0', '#8b5cf6', '#ddd6fe', '#0ea5e9', '#bae6fd'];
#statesColors;
#leadTimeColor = 'yellow';
Expand Down Expand Up @@ -893,5 +893,3 @@ class CFDRenderer extends UIControlsRenderer {

//endregion
}

export default CFDRenderer;
6 changes: 2 additions & 4 deletions src/graphs/control-chart/ControlRenderer.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import ScatterplotRenderer from '../scatterplot/ScatterplotRenderer.js';
import { ScatterplotRenderer } from '../scatterplot/ScatterplotRenderer.js';
import * as d3 from 'd3';

class ControlRenderer extends ScatterplotRenderer {
export class ControlRenderer extends ScatterplotRenderer {
color = '#0ea5e9';
timeScale = 'linear';
connectDots = false;
Expand Down Expand Up @@ -114,5 +114,3 @@ class ControlRenderer extends ScatterplotRenderer {
this.displayObservationMarkers(this.observations);
}
}

export default ControlRenderer;
6 changes: 2 additions & 4 deletions src/graphs/histogram/HistogramRenderer.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
import Renderer from '../Renderer.js';
import { Renderer } from '../Renderer.js';
import * as d3 from 'd3';

/**
* Class representing a Histogram graph renderer
*/
class HistogramRenderer extends Renderer {
export class HistogramRenderer extends Renderer {
#color = '#0ea5e9';
#padding = 3;
#binnedData;
Expand Down Expand Up @@ -295,5 +295,3 @@ class HistogramRenderer extends Renderer {

//endregion
}

export default HistogramRenderer;
4 changes: 1 addition & 3 deletions src/graphs/moving-range/MovingRangeGraph.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
class MovingRangeGraph {
export class MovingRangeGraph {
dataSet = [];

constructor(data) {
Expand Down Expand Up @@ -30,5 +30,3 @@ class MovingRangeGraph {
return Math.ceil(filteredData.reduce((acc, curr) => acc + curr.leadTime, 0) / filteredData.length);
}
}

export default MovingRangeGraph;
5 changes: 2 additions & 3 deletions src/graphs/moving-range/MovingRangeRenderer.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import ScatterplotRenderer from '../scatterplot/ScatterplotRenderer.js';
import { ScatterplotRenderer } from '../scatterplot/ScatterplotRenderer.js';
import * as d3 from 'd3';

class MovingRangeRenderer extends ScatterplotRenderer {
export class MovingRangeRenderer extends ScatterplotRenderer {
color = '#0ea5e9';
timeScale = 'linear';

Expand Down Expand Up @@ -98,4 +98,3 @@ class MovingRangeRenderer extends ScatterplotRenderer {
this.drawGraphLimits(this.currentYScale);
}
}
export default MovingRangeRenderer;
4 changes: 1 addition & 3 deletions src/graphs/scatterplot/ScatterplotGraph.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { calculateDaysBetweenDates } from '../../utils/utils.js';
/**
* Class representing a Scatterplot Graph Data
*/
class ScatterplotGraph {
export class ScatterplotGraph {
/**
* Creates a new ScatterplotGraph instance.
* @constructor
Expand Down Expand Up @@ -91,5 +91,3 @@ class ScatterplotGraph {
return dataSet;
}
}

export default ScatterplotGraph;
6 changes: 2 additions & 4 deletions src/graphs/scatterplot/ScatterplotRenderer.js
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
import { calculateDaysBetweenDates } from '../../utils/utils.js';
import UIControlsRenderer from '../UIControlsRenderer.js';
import { UIControlsRenderer } from '../UIControlsRenderer.js';
import styles from '../tooltipStyles.module.css';

import * as d3 from 'd3';

/**
* Class representing a Scatterplot graph renderer
*/
class ScatterplotRenderer extends UIControlsRenderer {
export class ScatterplotRenderer extends UIControlsRenderer {
color = '#0ea5e9';
currentXScale;
currentYScale;
Expand Down Expand Up @@ -654,5 +654,3 @@ class ScatterplotRenderer extends UIControlsRenderer {

//endregion
}

export default ScatterplotRenderer;
6 changes: 2 additions & 4 deletions src/graphs/scatterplot/SimpleScatterplotRenderer.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import ScatterplotRenderer from './ScatterplotRenderer.js';
import { ScatterplotRenderer } from './ScatterplotRenderer.js';

/**
* Class representing a Scatterplot graph renderer
*/
class SimpleScatterplotRenderer extends ScatterplotRenderer {
export class SimpleScatterplotRenderer extends ScatterplotRenderer {
currentXScale;
currentYScale;
timeScale = 'logarithmic';
Expand Down Expand Up @@ -84,5 +84,3 @@ class SimpleScatterplotRenderer extends ScatterplotRenderer {
percentile1 && this.drawHorizontalLine(y, percentile4, 'red', 'p4', '95%');
}
}

export default SimpleScatterplotRenderer;
4 changes: 1 addition & 3 deletions src/graphs/work-item-age/WorkItemAgeGraph.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { calculateDaysBetweenDates } from '../../utils/utils.js';
/**
* Class representing a Work Item Graph Data
*/
class WorkItemAgeGraph {
export class WorkItemAgeGraph {
constructor(data, states = ['analysis_active', 'analysis_done', 'in_progress', 'dev_complete', 'verification_start', 'delivered']) {
this.data = data;
this.states = states;
Expand Down Expand Up @@ -46,5 +46,3 @@ class WorkItemAgeGraph {
return ticketStates;
}
}

export default WorkItemAgeGraph;
Loading

0 comments on commit 41af796

Please sign in to comment.