kuhumcst / korp-setups

Docker setups for all Korp installations maintained by NorS.
MIT License
1 stars 0 forks source link

Tilføjer "threats"-mode. #12

Closed phildiderichsen closed 3 years ago

phildiderichsen commented 3 years ago
phildiderichsen commented 3 years ago

Hey

Godt nytår! :)

Yes, det kunne man godt kigge lidt på.

Kunne du gøre mig en tjeneste og prøve at bygge threats-setuppet lokalt? (Kræver at du er på vpn for at kunne hente dataene).

Bemærk i øvrigt: Jeg tror ikke sync.sh vil virke uden lokalt frontend-volume. Funktionaliteten kan nogenlunde replikeres ved at bygge setup-imaget forfra - det tager lidt længere tid, men det er ikke så slemt. Spørgsmålet er om sync.sh kan/bør fjernes.

:) P

Den 5. jan. 2021 kl. 11.49 skrev Simon Gray notifications@github.com:

 @simongray approved this pull request.

Ser umiddelbart fint ud! Fedt at du kun behøvede tilføje den ene linje med subversion til backend-Dockerfilen. Det tyder jo på at den arkitektur fungerer nogenlunde for os begge.

Det eneste der springer mig i øjnene er, at jeg ville ønske vi havde en løsning til at at genbruge mere fra en fælles common.js og config.js, men det har ikke noget specifikt med dit PR at gøre.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

simongray commented 3 years ago

Godt nytår selv 🙏

Det kan sagtens være at sync.sh bør flyttes til mit eget image. Men så igen, det skader jo ikke at det er der. Det kan være der kan indbygge et tjek og en relevant fejlbesked i tilfælde hvor det ikke vil virke.

Jeg skal nok teste dit setup, man derfor kan du sagtens merge alligevel.