A few minor fixes - Githubissues

Altinn / app-localtest

Solution for testing apps locally on your own machine

8 stars 14 forks source link

A few minor fixes #36

Closed ivarne closed 1 year ago

ivarne commented 1 year ago

Use cached layers for Dockerfile to improve repeated build times locally.
Set longer expiration on the cookie that remembers your preferred user for login (as well as syncing AltinnPartyId)
Ignore a few nullability warnings to make the build warning free
- I don't have any idea about better error handling than a NullPointerException so I just ignore with !

Verification

[x] Your code builds clean without any errors or warnings
[ ] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[ ] All tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

ivarne commented 1 year ago

@TheTechArch You seem to be the one merging into app-localtest recently. Any feedback on these small changes?