Skip to content

Conversation

@thoalm
Copy link
Contributor

@thoalm thoalm commented Oct 28, 2025

💰 Hva forsøker du å løse i denne PR'en

Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-26645

Lager en egen hook useFeatureToggles for å ta i bruk feature toggles. Tar ut tilstanden fra AppContext og lar react-query håndtere den.

🔎️ Er det noe spesielt du ønsker å fremheve?

Nei

✅ Checklist

  • Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️
  • Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇

🤷‍♀ ️Hvor er det lurt å starte?

Alt i ett.

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

👀 Screen shots

Ingen visuelle endringer.

@thoalm thoalm requested a review from a team as a code owner October 28, 2025 12:18
@thoalm thoalm force-pushed the NAV_26645_lag_use_toggles_hook branch 4 times, most recently from 485756f to 5e82730 Compare October 30, 2025 08:58
Copy link
Contributor

@GardOS GardOS left a comment

Choose a reason for hiding this comment

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

Ser bra ut ✅

Støtter navnendring fra ToggleNavn til Toggle 🏅

Også får vi diskutere på utviklerpraten hvor smart det er å ha fallback på at alle toggles blir skrudd av ved feil

@thoalm thoalm force-pushed the NAV_26645_lag_use_toggles_hook branch 2 times, most recently from 6df1f04 to f4d54e6 Compare November 6, 2025 08:16
@thoalm thoalm changed the title NAV-26645: Legger til en egen useToggles hook for feature toggles NAV-26645: Legger til en egen useFeatureToggles hook for feature toggles Nov 7, 2025
@thoalm thoalm force-pushed the NAV_26645_lag_use_toggles_hook branch from b7384d1 to 5479a10 Compare November 11, 2025 11:12
@thoalm thoalm force-pushed the NAV_26645_lag_use_toggles_hook branch 4 times, most recently from b8b241d to 7d27e82 Compare December 11, 2025 14:48
@thoalm thoalm force-pushed the NAV_26645_lag_use_toggles_hook branch 2 times, most recently from 70b705c to e8a7ff1 Compare January 5, 2026 15:02
@thoalm thoalm force-pushed the NAV_26645_lag_use_toggles_hook branch from e8a7ff1 to c98c219 Compare January 5, 2026 15:21
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.

3 participants