Skip to content

Conversation

@sirimykland
Copy link
Collaborator

No description provided.

@sirimykland sirimykland requested a review from a team as a code owner October 28, 2025 16:25
@sirimykland sirimykland force-pushed the feature/legg-til-app-versjon-i-server branch 2 times, most recently from e7393a6 to f926cd2 Compare October 28, 2025 16:28
@sirimykland sirimykland force-pushed the feature/legg-til-app-versjon-i-server branch from f926cd2 to f7328dc Compare October 28, 2025 16:32
@sonarqubecloud
Copy link

Copy link
Collaborator

@tor-nav tor-nav left a comment

Choose a reason for hiding this comment

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

Nyttig!

const [serverVersjon, setServerVersjon] = useState<string | null>(null);
const [nyVersjonTilgjengelig, setNyVersjonTilgjengelig] = useState(false);

const [klientVersjon, setKlientVersjon] = useState<string | null>(null);
Copy link
Collaborator

Choose a reason for hiding this comment

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

dette er pirk på høgt nivå, men for meg hadde det vore meir naturleg at linje 7 og 9 byttar plass

const visLos = (e: React.SyntheticEvent) => {
if (e.type === 'click') {
void navigate('/');
window.location.reload();
Copy link
Collaborator

Choose a reason for hiding this comment

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

og eit par sonar-varsel på å bruka globalThis i staden for window

"DekoratorMedFeilviserSakIndex.EndreTilLysTema": "Endre til lyst tema",
"DekoratorMedFeilviserSakIndex.EndreTilMorkTema": "Endre til mørkt tema"
"DekoratorMedFeilviserSakIndex.EndreTilMorkTema": "Endre til mørkt tema",
"DekoratorMedFeilviserSakIndex.NyVersjon": "Det finnes en ny versjon av applikasjonen. Klikk for å laste siden på nytt."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Kanskje si fra hvis man mister arbeidet (det man har skrevet i begrunnelse / brev)

</InternalHeader.Title>
<Spacer />
{nyVersjonTilgjengelig && (
<InternalHeader.Button onClick={() => window.location.reload()}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Blir Button synlig når man står inne i en åpen sak? Risiko for å miste arbeidet?
Gjør reload at man blir stående med aktuell sak / behandling eller havner man annet sted ?
Tenker det er nyttig at vi er tydelige dersom man evt mister arbeid.

@sirimykland sirimykland marked this pull request as draft October 30, 2025 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants