Skip to content

Conversation

azclasen
Copy link
Member

@azclasen azclasen commented Oct 10, 2026 •

Målet var å erstatte mock-data på Techniques-siden med ekte data fra databasen gjennom GraphQL-backenden.
Først satte jeg opp Apollo Client. Den fungerer som bindeleddet mellom React og GraphQL-serveren.

Alt som ble gjort:

  • Installerte: apollo/client, konfiguerte, ApolloClient & ApolloProvider
  • Opprettet GraphQL-query. Brukte useQuery() for å hente data
  • Erstattet mock-data
  • Verifiserte hele kjeden

Det som gjenstår:

  • Det er kun mitreID, name og description som hentes, mangler dermed tactic og platform, men det er litt mer komplisert og må gjøres senere ved å joine tabeller

KI bruk:

  • fikk informasjon om hvordan og hvorfor installere og bruke Apollo, og konfigurasjonen knyttet til det
  • den viste til hvilke filer som må endres (i både frontend og backend)
  • når jeg fikk feilmeldinger og andre feil som når nettsiden ikke svarte lenger, så hjalp den meg med feilsøking og løsninger

Closes #43

@azclasen azclasen requested a review from a team as a code owner October 10, 2026 21:14
nikitar
nikitar previously approved these changes Oct 12, 2026
Copy link
Member

@nikitar nikitar left a comment

Choose a reason for hiding this comment

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

Smoooth

@azclasen azclasen dismissed nikitar’s stale review October 12, 2026 07:09

The merge-base changed after approval.

nikitar
nikitar previously approved these changes Oct 12, 2026
@azclasen azclasen dismissed nikitar’s stale review October 12, 2026 07:16

The merge-base changed after approval.

Copy link
Member

@nikitar nikitar left a comment

Choose a reason for hiding this comment

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

Hei Alea, opprettet du connect-techniques-frontend-backend, opprettet du den fra main eller fra techniques-backend?

#47

Screenshot 2026-10-12 at 09 16 21

#48

Screenshot 2026-10-12 at 09 16 38

Jeg ser at denne PR inneholder også commit fra den forrige PR #47, dermed nå som jeg fikk godkjenne #47, så overlapper #47 med #48 ---> jeg får ikke godkjenne #48 :(

Kan du sjekke om connect-techniques-frontend-backend branchen er ferdig oppdatert, og eventuelt synkronisere den med nyeste main?

Når alt er klart, kan du si fra, så godkjenner jeg PR #48 på nytt. Da bør vi kunne merge den,

@azclasen azclasen force-pushed the connect-techniques-frontend-backend branch from 6b8fde4 to 5e04f56 Compare October 12, 2026 11:03
Copy link
Member

@nikitar nikitar left a comment

Choose a reason for hiding this comment

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

Tip top

@nikitar nikitar merged commit e3966ac into main Oct 12, 2026
@nikitar nikitar changed the title Connect techniques frontend backend Erstatte mock-data med database-data på Techniques page Oct 18, 2026
@nikitar nikitar added the enhancement New feature or request label Oct 18, 2026
Sign in to join this conversation on GitHub.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Koble frontend til backend med Techniques-queries
2 participants