change for use of git links for images #301
Conversation
|
ser ihvertfald fint alle disse, på en ny addresse jeg satte op :) |
|
Han spørger også om vi kan gøre det valgfrit at hente dem. Det kunne man gøre ved at:
Der er sikkert noget mere der skal laves i HA integrationen, men det må jeg kigge på. |
|
Nej, han beder om at gøre det valgfrit at lægge de lange base64 strings ind i databasen hver gang sensoren ændrer en attribut som sker nu, hvilket dette løser. Dette er den rigtige løsning i alle tilfælde IMHO, da HA selv cacher billederne lokalt fra den slags links. Hvis noget kunne man argumentere for at linket skulle være til releasen der passer til, så linket ikke er det samme hvis vi potentielt ændrer nogle af billederne ved en release. Den anden løsning han skitserer er at have links til filerne lokalt, der kræver at brugeren selv downloader og lægger korrekt i www folderen i HA. Hvis en bruger virkelig vil have det kan det gøres, men forskellen til de trods alt relativt korte links er minimal i den sammenhæng vil jeg sige... |
|
Ok, så har jeg læst det forkert. Men så synes jeg vi holder fast i den løsning du har lavet nu, og lukker den der. Er der senere brug for ændringer alligevel, må vi kigge på det. |
|
Nu vi har en beta ude som vi jo gerne vil lukke, tænker jeg vi venter med den her til efter den release og så kan det komme med i den næste med også det her opdatering af next_pickup i løbet af dagen når det er blevet testet lidt... |
|
Aftale |
I think this works fine, and would solve #230