-
Notifications
You must be signed in to change notification settings - Fork 0
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
Visning av kundeansvalrlig og bugfixes #561
Conversation
const customerEmployees = employeesCustomer.filter( | ||
(ec) => ec.customer === customer | ||
) | ||
const employeesLastPeriod = customerEmployees |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Før var antall ansatte, anta unike over 3 uker, mens antall timer var forige uke.
Nå deler vi opp begge datapunktene i enten 1 uke eller 4 uker og sender alt til frontenden.
const project_data = JSON.parse(perProject) | ||
const customer_data = JSON.parse(employeeCustomers) | ||
const aggregatedData = createCustomerCardData(project_data, customer_data) | ||
const accountManagerTable = JSON.parse(accountManager) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
legger også til account manager info
@@ -27,9 +24,21 @@ export const useHoursBilledPerCustomerCharts = () => | |||
revalidateOnFocus: false, | |||
}) | |||
|
|||
export const useAllCustomerData = (): CustomerData[] => { | |||
export const useAllCustomerData = (): CustomerCardData[] => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vi har ikke antall konsulenter og timer for historiske kunder. Dette burde egentlig fikses. Kansje en forbedring er å hente historisk data sammen med customerCards også heller filtrere ut manglende konsulenter i front
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dette har jeg nå fikset ved å ta med historiske kunder i customerCard apiet og filtrere dem ut senere
/> | ||
</GridItemContent> | ||
</GridItem> | ||
) | ||
} | ||
|
||
export default CustomerCard | ||
|
||
const ComponentRoot = styled(Grid, { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
flytter styled komponentene nederst i fila for å rydde litt
@@ -43,4 +43,5 @@ export default function OrganizationStructurePage() { | |||
|
|||
const RelativePositionedBox = styled('div')({ | |||
position: 'relative', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fikser så grafen tar den plassen den trenger og dytter footern under bildet
No description provided.