Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Velge å kun få dagsrapporter #131

Closed
josteinaj opened this issue Sep 28, 2018 · 7 comments
Closed

Velge å kun få dagsrapporter #131

josteinaj opened this issue Sep 28, 2018 · 7 comments
Assignees
@espso
Labels
forbedring Middels jobb Aktivitet som gjennomføres i løpet av en periode, og tar mer enn en uke å gjøre Middels prioritet

Comments

@josteinaj
Copy link
Member

Det bør være mulig å velge å kun få dagsrapporter, istedenfor å få en e-post for hver eneste bok.
@josteinaj josteinaj added forbedring Middels jobb Aktivitet som gjennomføres i løpet av en periode, og tar mer enn en uke å gjøre Middels prioritet labels Sep 28, 2018
@espso
Copy link
Contributor

espso commented Oct 2, 2018

produksjonssystem.yaml:

nlbpub-to-pef:
  - daily:
     - [email protected]
  [email protected]

Er dette en grei løsning? Hvis man kun vil ha dagsrapport kan vi etter hvert legge de til i daily. Tenker uansett at dagsrapportene bør finpusses litt først. Underkonfigs i email_settings oppdateres allerede og er lagret under config["daily"] i hver pipeline.

@josteinaj
Copy link
Member Author

Det ser ut som en grei løsning ja.

En idé: hva om vi lagrer HTMLen fra e-posten i en egen email.html-fil i samme mappe som log.txt? Da kunne man lenket til den fra dagsrapporten...

@espso
Copy link
Contributor

espso commented Oct 2, 2018

Altså mailene du snakker om er de vanlige mailene? Hvis man vil hente mer info liksom? Tror det vil funke bra ja

@josteinaj
Copy link
Member Author

Ja.

espso added a commit that referenced this issue Oct 5, 2018
@espso
Daily mail - bugfixes, #131 and links to mail saved in logs
f9396b0
@josteinaj
Copy link
Member Author

@espso: Se også: nlbdev/pipeline#204 (comment)

@espso
Copy link
Contributor

espso commented Oct 15, 2018

Ikke helt sikker på om det fungerer å kun få dagsrapporter eller ikke. Uansett så ser det ut som tråden har slitt litt i helgen, muligens fordi serveren har vært helt makset ut på konverteringer. Før så har jeg tenkt at den bare skal sende mellom 7-9, men kanskje det kan være greit å sende når som helst, og lagre htmlen i dagsrapport mappa etter at den er sendt.

@espso
Copy link
Contributor

espso commented Oct 16, 2018

Ser ut som det fungerer som det skal @josteinaj? Hvis du ser på dagsrapport for kontroll av punktskrift er det ingen linker til mail fordi det ikke er laget noen mail (ingen mottakere). Kanskje en av oss skal stå som mottaker eller gjøre det på en annen måte. Lager en annen sak på forbedring av dagsrapporter

@espso espso closed this as completed Oct 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@espso espso

Labels
forbedring Middels jobb Aktivitet som gjennomføres i løpet av en periode, og tar mer enn en uke å gjøre Middels prioritet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@josteinaj @espso