Skip to content

Conversation

xeniag
Copy link
Member

@xeniag xeniag commented Nov 9, 2026

  • Changed default API URL from '/graphql' to '/project2/graphql' in Apollo client.
  • Updated Vite configuration to add a proxy for '/project2/graphql' to forward requests to the backend.

- Changed default API URL from '/graphql' to '/project2/graphql' in Apollo client.
- Updated Vite configuration to add a proxy for '/project2/graphql' to forward requests to the backend.
@xeniag xeniag marked this pull request as ready for review November 9, 2026 17:52
@xeniag xeniag requested a review from a team as a code owner November 9, 2026 17:52
Copy link
Member

@ronjalh ronjalh left a comment

Choose a reason for hiding this comment

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

Speed Claude review, dobbeltsjekk det så burde det være good:
Det er likevel tre ting jeg ville kommentert på PR-en:
'/graphql' er nå overflødig. Hvis klienten alltid bruker /project2/graphql, treffer ingen den gamle regelen. Den kan fjernes, ellers ser det ut som det finnes to gyldige stier.
/images er ikke flyttet. Hvis bilder også går via /project2/ på VM-en, vil de virke lokalt, men gi 404 i produksjon, eller omvendt. Det bør behandles likt som graphql.
/project2 er hardkodet. Det er ryddigere å sette prefikset ett sted, nemlig base i Vite:
// vite.config.ts
base: '/project2/',

Copy link
Member

@ronjalh ronjalh left a comment

Choose a reason for hiding this comment

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

Maybe consider adding 4000 as port to the backend tests as well and maybe not have the 200 line additions in package lock that I dont really get

Copy link
Member

Choose a reason for hiding this comment

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

Is this relevant?

xenia_grib and others added 3 commits November 10, 2026 20:25
The lockfile changes in 1cfdae8 were unrelated dependency bumps
(no package.json changed), so the lockfile now matches main again.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Matches the backend's new default port from f5c286a.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copy link
Member

@ronjalh ronjalh left a comment

Choose a reason for hiding this comment

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

lgtm

Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants