Skip to content

Enable create#323

Merged
justinorringer merged 3 commits intoRedHatInsights:masterfrom
OlhaTmlk:enable-create
Apr 28, 2026
Merged

Enable create#323
justinorringer merged 3 commits intoRedHatInsights:masterfrom
OlhaTmlk:enable-create

Conversation

@OlhaTmlk
Copy link
Copy Markdown
Contributor

@OlhaTmlk OlhaTmlk commented Apr 15, 2026

Description

  • Enable create, import, and duplicate dashboard actions across all pages (Dashboard Hub, generic dashboard page, and landing page)
  • Centralize dashboard state management by moving CRUD operations into Jotai atoms (createDashboardAtom, importDashboardAtom, duplicateDashboardAtom, setDefaultDashboardAtom) so dashboards auto-refresh after mutations
  • Refactor DuplicateModal to read dashboards from atom instead of props, and add preselectedDashboardId support for context-aware duplication from the dashboard table
  • Fix tests to account for atom-based state

RHCLOUD46796


Screenshots

Screenshot 2026-04-15 at 11 13 31 Enabled create actions
Screenshot 2026-04-15 at 11 13 45 Tooltip for duplicate button in dashboard hub
Screenshot 2026-04-21 at 12 43 04 Enabled create/edit actions in generic page

@OlhaTmlk
Copy link
Copy Markdown
Contributor Author

/retest

@OlhaTmlk OlhaTmlk marked this pull request as ready for review April 21, 2026 11:03
@OlhaTmlk OlhaTmlk requested a review from a team as a code owner April 21, 2026 11:03
Copy link
Copy Markdown
Contributor

@justinorringer justinorringer left a comment

Choose a reason for hiding this comment

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

Looking good! A couple little things plus lint

Comment thread src/Components/Header/Header.tsx
Comment thread src/hooks/useDuplicateDashboard.ts Outdated
@OlhaTmlk OlhaTmlk requested a review from justinorringer April 28, 2026 09:26
Copy link
Copy Markdown
Contributor

@justinorringer justinorringer left a comment

Choose a reason for hiding this comment

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

LGTM!

@justinorringer justinorringer merged commit 11ad526 into RedHatInsights:master Apr 28, 2026
10 checks passed
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