lamassu/lamassu-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: analytics graph memoizing

Browse source
This commit is contained in:
Sérgio Salgado 2022-02-10 18:09:39 +00:00
parent 4015e23d34
commit 35c81ce112
3 changed files with 12 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

2
new-lamassu-admin/src/components/inputs/base/Select.styles.js
View file

@ -14,7 +14,7 @@ export default {
},
select: {
width: WIDTH,
zIndex: 2,
zIndex: 1,
'& label': {
extend: label1,
color: offColor,

6
new-lamassu-admin/src/pages/Analytics/graphs/HourOfDayBarGraph.js
View file

@ -21,7 +21,8 @@ const Graph = ({
timezone,
setSelectionCoords,
setSelectionData,
setSelectionDateInterval
setSelectionDateInterval,
selectedMachine
}) => {
const ref = useRef(null)
@ -428,5 +429,6 @@ export default memo(
Graph,
(prev, next) =>
R.equals(prev.period, next.period) &&
R.equals(prev.selectedDay, next.selectedDay)
R.equals(prev.selectedDay, next.selectedDay) &&
R.equals(prev.selectedMachine, next.selectedMachine)
)

9
new-lamassu-admin/src/pages/Analytics/graphs/TopMachinesBarGraph.js
View file

@ -11,7 +11,7 @@ import {
fontSecondary
} from 'src/styling/variables'
const Graph = ({ data, machines, currency }) => {
const Graph = ({ data, machines, currency, selectedMachine }) => {
const ref = useRef(null)
const AMOUNT_OF_MACHINES = 5
@ -304,4 +304,9 @@ const Graph = ({ data, machines, currency }) => {
return <svg ref={ref} />
}
export default memo(Graph, (prev, next) => R.equals(prev.period, next.period))
export default memo(
Graph,
(prev, next) =>
R.equals(prev.period, next.period) &&
R.equals(prev.selectedMachine, next.selectedMachine)
)