Skip to content

Implemented frontend optimalisations#196

Merged
stijnpotters1 merged 8 commits intomasterfrom
fix/implement-services
Feb 4, 2026
Merged

Implemented frontend optimalisations#196
stijnpotters1 merged 8 commits intomasterfrom
fix/implement-services

Conversation

@stijnpotters1
Copy link
Contributor

@stijnpotters1 stijnpotters1 commented Jan 29, 2026

  • Now uses services instead of api calls inside components, also i prevented api calls in useEffects to prevent duplicate requests
  • fixed consistent navigation
  • fixed studio pallette bug
  • studio structure also is updated and fetched properly
  • added consistent loading states
  • added loading spinner

@stijnpotters1 stijnpotters1 marked this pull request as ready for review January 29, 2026 14:28
Copy link
Member

@philipsens philipsens left a comment

Choose a reason for hiding this comment

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

Please look at the warnings in the build log (Lint frontend): https://github.com/frankframework/flow/actions/runs/21482043866/job/61880755636?pr=196#step:7:17

Matthbo
Matthbo previously requested changes Feb 3, 2026
@stijnpotters1 stijnpotters1 requested a review from Matthbo February 3, 2026 17:43
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 3, 2026

@stijnpotters1 stijnpotters1 dismissed Matthbo’s stale review February 4, 2026 12:01

Already approved by daan

@stijnpotters1 stijnpotters1 merged commit 1a07ed3 into master Feb 4, 2026
5 checks passed
@stijnpotters1 stijnpotters1 deleted the fix/implement-services branch February 4, 2026 12:01
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.

Studio Context is broken Navigation still not working consistent Use services instead of making api calls inside components

4 participants