Skip to content

chore: add module block with analytics and supportCaseData to deploy config#759

Open
dajohnso wants to merge 3 commits intoRedHatInsights:masterfrom
dajohnso:add-analytics-keys
Open

chore: add module block with analytics and supportCaseData to deploy config#759
dajohnso wants to merge 3 commits intoRedHatInsights:masterfrom
dajohnso:add-analytics-keys

Conversation

@dajohnso
Copy link
Copy Markdown

Summary

  • Adds the entire module block to frontend.yaml including analytics keys, manifestLocation, supportCaseData config, and module routes
  • Aligns with the pattern used by other frontends (inventory, advisor, tasks, ocp-advisor)

Test plan

  • Verify the YAML is valid and the deploy CRD parses correctly
  • Confirm analytics events are sent in stage with the correct API key

- Add @project-kessel/react-kessel-access-check and @tanstack/react-query dependencies
- Wrap app with QueryClientProvider and AccessCheck.Provider when Kessel is enabled
- Refactor PermissionsProvider to support dual-path (RBAC v1 vs Kessel v2)
- Create useKesselWorkspaces utility hook for workspace fetching
- Simple approach: show all cards if user has any workspaces
- Feature flag: hbi.kessel-migration
- Add jest mock for Hooks module to provide useFeatureFlag context
- Fixes test failures caused by missing Unleash feature flag context
- All 3 App.test.js tests now passing
@dajohnso dajohnso requested a review from a team as a code owner April 22, 2026 19:38
@mtclinton
Copy link
Copy Markdown

@dajohnso think you might need to rebase this one? Claude sometimes has issues with that, so you might have to manually update branch to continue?

Copy link
Copy Markdown

@mtclinton mtclinton left a comment

Choose a reason for hiding this comment

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

See above, PR needs a rebase :)

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.

2 participants