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.

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

Sett tidssone på godkjent_av_arbeidsgiver #128

Merged
merged 2 commits into from
Sep 13, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,12 +1,16 @@
package no.nav.arbeidsgiver.tiltakrefusjon.hendelseslogg

import jakarta.persistence.*
import jakarta.persistence.Convert
import jakarta.persistence.Entity
import jakarta.persistence.EnumType
import jakarta.persistence.Enumerated
import jakarta.persistence.Id
import no.nav.arbeidsgiver.tiltakrefusjon.refusjon.BrukerRolle
import no.nav.arbeidsgiver.tiltakrefusjon.utils.Now
import no.nav.arbeidsgiver.tiltakrefusjon.utils.ulid
import org.hibernate.annotations.JdbcTypeCode
import org.hibernate.type.SqlTypes
import java.time.LocalDateTime
import java.time.Instant

@Entity
data class Hendelseslogg(
Expand All @@ -23,5 +27,5 @@ data class Hendelseslogg(
) {
@Id
val id: String = ulid()
val tidspunkt: LocalDateTime = Now.localDateTime()
val tidspunkt: Instant = Now.instant()
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@ package no.nav.arbeidsgiver.tiltakrefusjon.hendelseslogg

import no.nav.arbeidsgiver.tiltakrefusjon.refusjon.BrukerRolle
import no.nav.arbeidsgiver.tiltakrefusjon.refusjon.erGyldigFnr
import java.time.LocalDateTime
import java.time.Instant

data class HendelsesloggDTO(
val refusjonId: String,
val korreksjonId: String?,
val utførtAv: String,
val event: String,
val metadata: HendelseMetadata? = null,
val tidspunkt: LocalDateTime,
val tidspunkt: Instant,
) {
constructor(hendelseslogg: Hendelseslogg) : this(
refusjonId = hendelseslogg.refusjonId,
Expand Down
2 changes: 1 addition & 1 deletion src/main/resources/application-dev-gcp-labs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,4 +46,4 @@ tiltak-refusjon:
norg:
fake: true
beslutter-ad-gruppe:
id: BESLUTTER_AD_GRUPPE
id: 1a1d2745-952f-4a0f-839f-9530145b1d4a
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
-- godkjent_av_arbeidsgiver-feltet har ikke tidssone i databasen, men er en instant i jpa-entiteten.
-- Instants er tidspunkter med tidssoner, så kolonnen burde også ha tidssone.
alter table refusjon alter column godkjent_av_arbeidsgiver type timestamp with time zone
using godkjent_av_arbeidsgiver at time zone 'UTC';
Oddsor marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
-- godkjent_av_arbeidsgiver-feltet har ikke tidssone i databasen, men er en instant i jpa-entiteten.
-- Instants er tidspunkter med tidssoner, så kolonnen burde også ha tidssone.
alter table refusjon alter column utbetalt_tidspunkt type timestamp with time zone
using utbetalt_tidspunkt at time zone 'UTC';
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
-- tidspunkt-feltet har ikke tidssone i databasen, og er en "localdatetime" i jpa-entiteten.
-- For å få tidspunktene til å bli riktige MED tidssone må vi derfor bruke "at time zone" med
-- europe/oslo i stedet for utc, slik vi gjorde med feltene hvor jpa-entiteten bruker instant.
alter table hendelseslogg alter column tidspunkt type timestamp with time zone
using tidspunkt at time zone 'Europe/Oslo';
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obs obs, merk forskjellen her!

2 changes: 2 additions & 0 deletions src/test/resources/application-dockercompose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,8 @@ tiltak-refusjon:
fake: true
varsling:
varsling-klar-cron: "0 */5 0 17 * ?"
beslutter-ad-gruppe:
id: 1a1d2745-952f-4a0f-839f-9530145b1d4a
server:
port: 8081

Expand Down