tanettrimas/finn-pris-per-kvadratmeter

Er du lei av å måtte manuelt regne ut kvadratmeterprisen når du leter etter drømmeboligen? Bruk denne til å regne det ut for deg!

finn-pris-per-kvadratmeter's Issues

forenkle automatisert release

Dagens oppsett, ref #9 bruker nå dependencies og fetch kalls direkte i koden. Dette kan forenkles ved å bruke https://extensionworkshop.com/documentation/develop/web-ext-command-reference/ og https://www.npmjs.com/package/chrome-webstore-upload-cli

automatisere framtidige publiseringer

Chrome Web Store har et api som gjør det mulig å oppdatere en extension og publisere den, f.eks som en hook etter en push/pr til master branchen.

https://developer.chrome.com/webstore/using_webstore_api

Legge til antall minutter inn til sentrum

Tungvindt å sjekke reisetid manuelt med Google Maps, kan f.eks bruke diverse reise-apier for å hente data basert på gateadressen og finne ut cirka hvor langt det tar å reise inn til sentrum.

browser is not defined

Hei, takk for kul extension! Men den fungerer ikke etter den nye versjonen hos meg:

implementer extension på flere sider

Per nå fungerer kun extension på boliger til salgs-siden. Finn har lik oppsett på de aller fleste boligannonsesidene, så det skal være ganske enkelt å få implementert slik at det kan gjøres på flere sider her.

Action Required: Fix Renovate Configuration

There is an error with this repository's Renovate configuration that needs to be fixed. As a precaution, Renovate will stop PRs until it is resolved.

File: renovate.json
Error type: The renovate configuration file contains some invalid settings
Message: Invalid configuration option: draft

Lytte til DOM-endringer på en annen måte

Per nå brukes MutationObserver til å lytte til om DOMen endrer seg ved at kort blir fjernet eller lagt til, og da på en grense på minst ti stk. Mulig dette kan gjøres litt smidigere ved å implementere dette i en background-script f.eks.

Action Required: Fix Renovate Configuration

There is an error with this repository's Renovate configuration that needs to be fixed. As a precaution, Renovate will stop PRs until it is resolved.

Location: renovate.json
Error type: Invalid JSON (parsing failed)
Message: Syntax error: expecting String near .json", }

Testing / Continous Integration

Repoet har for øyeblikket ingen tester, hvilket gjør det vanskelig å finne ut om koden faktisk vil fungere hvis Finn plutselig finner ut at de skal endre html strukturen.

Dette er et par ting som bør ordnes:

• Hente et par seksjoner i HTML fra finn via puppeter
• Inspecte om ting endrer seg som det skal (pris blir satt, formattert riktig, riktig priskalkulering etc)

Testene bør kunne kjøres ved hver merge/push til master

implementer enkel modularisering

Per nå er koden skrevet i èn js-fil, og det er ikke særlig skalerbart. Med tanke på at man kan bruke en bundler, slik at man bruke npm og dra nytte av tredjepartsbibloteker. Dermed skal man kun behøve å peke på en bundlet fil i chrome extension.

priskalkulering bruker ikke totalpris

Det er mange som er interessert å ta utgangspunkt i totalpris når man ser på pris per kvadratmeter og ikke nødvendigvis prisantydning som det foreligger i dagens løsning.

Endringsforslag: Dersom det finnes totalpris fremvist på siden, så bruk denne i stedet.

TypeScript

For bedre type-checking og "development experience", kan det vurderes om prosjektet skal skrives om til TypeScript.