MIM-178: Do not count terminated leases as active

[henrikhenrikpersson]

@mittistormen Kan det här ställa till något för er?

[linear]

MIM-178 Exkludera makulerade kontrakt

[mittistormen]

@mittistormen Kan det här ställa till något för er?

Det verkar som att isLeaseActive först har skrivits av @johanneskarlsson när vi jobbade med externa bilplatser förra vintern (det är i produktion sedan i våras). Se https://dev.azure.com/mimeronline/ONECore/_workitems/edit/1176
Men det verkar som att funktionen används av de flesta funktioner i hela adaptern för att hämta kontrakt mot xpand.
När det gäller interna bilplatser så använder vi kontrakten för att göra urval/besluta vem som ska få erbjudande på kontrakt.

Tror vi behöver blanda in david i detta för att definiera vad som räknas som aktiva kontrakt och inte

[henrikhenrikpersson]

@mittistormen Kan det här ställa till något för er?

Det verkar som att isLeaseActive först har skrivits av @johanneskarlsson när vi jobbade med externa bilplatser förra vintern (det är i produktion sedan i våras). Se https://dev.azure.com/mimeronline/ONECore/_workitems/edit/1176 Men det verkar som att funktionen används av de flesta funktioner i hela adaptern för att hämta kontrakt mot xpand. När det gäller interna bilplatser så använder vi kontrakten för att göra urval/besluta vem som ska få erbjudande på kontrakt.

Tror vi behöver blanda in david i detta för att definiera vad som räknas som aktiva kontrakt och inte

Bakgrund till detta PR är att vi inte vill visa makulerade kontrakt i Odoo, och vi (inklusive David) kom fram till att vi överhuvudtaget inte vill använda oss av makulerade kontrakt nånstans i onecore. Vid behov går det fortfarande att inkludera alla kontrakt med includeTerminatedLeases-queryparametern.

[mittistormen]

@mittistormen Kan det här ställa till något för er?

Det verkar som att isLeaseActive först har skrivits av @johanneskarlsson när vi jobbade med externa bilplatser förra vintern (det är i produktion sedan i våras). Se https://dev.azure.com/mimeronline/ONECore/_workitems/edit/1176 Men det verkar som att funktionen används av de flesta funktioner i hela adaptern för att hämta kontrakt mot xpand. När det gäller interna bilplatser så använder vi kontrakten för att göra urval/besluta vem som ska få erbjudande på kontrakt.
Tror vi behöver blanda in david i detta för att definiera vad som räknas som aktiva kontrakt och inte

Bakgrund till detta PR är att vi inte vill visa makulerade kontrakt i Odoo, och vi (inklusive David) kom fram till att vi överhuvudtaget inte vill använda oss av makulerade kontrakt nånstans i onecore. Vid behov går det fortfarande att inkludera alla kontrakt med includeTerminatedLeases-queryparametern.

Om det är så att vi inte någonstans i onecore vill använda oss av makulerade kontrakt så låter det ju rätt och riktigt att göra den här förändringen :)