Skip to content

chore(atomic): migrate atomic-quickview-modal to Lit#6949

Merged
alexprudhomme merged 25 commits intomainfrom
copilot/migrate-atomic-quickview-modal
Jan 27, 2026
Merged

chore(atomic): migrate atomic-quickview-modal to Lit#6949
alexprudhomme merged 25 commits intomainfrom
copilot/migrate-atomic-quickview-modal

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 12, 2026 •
edited by alexprudhomme
Loading

✅ Checklist

  • 🧪 The component is unit tested
  • 🧪 The component includes E2E tests
  • 🗑️ Old Cypress tests exclusive to the component are removed
  • 📖 The component is documented in storybook with an .mdx file
  • ♿ The component complies with the Web Content Accessibility Guidelines.
  • 🌐 All strings intended for humans or assistive technology must be localized with i18n.
  • 📦 The Lit component is exported in the appropriate index.ts and lazy-index.ts files.
  • 🎨 CSS parts are documented still accessible.
  • 🦥 Slotted Content, public methods and properties are documented
  • 🔄 The component outputs the same Angular output as before with Stencil
  • 🏷️ The component declares the component type in the HTMLElementTagNameMap

https://coveord.atlassian.net/browse/KIT-4980

Copilot AI and others added 3 commits January 12, 2026 20:46
@alexprudhomme
Migrate atomic-quickview-modal component to Lit
acee7a5 
Co-authored-by: alexprudhomme <78121423+alexprudhomme@users.noreply.github.com>
@alexprudhomme
Add comprehensive unit tests for atomic-quickview-modal
c96f164 
Co-authored-by: alexprudhomme <78121423+alexprudhomme@users.noreply.github.com>
@alexprudhomme
Complete atomic-quickview-modal migration to Lit
c30069a 
Co-authored-by: alexprudhomme <78121423+alexprudhomme@users.noreply.github.com>
Copilot AI changed the title [WIP] Migrate atomic quickview modal component feat(atomic): migrate atomic-quickview-modal to Lit Jan 12, 2026
Copilot AI requested a review from alexprudhomme January 12, 2026 20:53
@alexprudhomme alexprudhomme changed the title feat(atomic): migrate atomic-quickview-modal to Lit chore(atomic): migrate atomic-quickview-modal to Lit Jan 12, 2026
@alexprudhomme alexprudhomme added new-lit-component chromatic Opt-out of Chromatic testing labels Jan 12, 2026
Copilot AI temporarily deployed to Prerelease (CDN) January 13, 2026 18:51 Inactive
github-code-quality[bot]
github-code-quality Bot found potential problems Jan 26, 2026
View reviewed changes
Comment thread packages/atomic/src/components/search/atomic-quickview-modal/atomic-quickview-modal.spec.ts Fixed
@alexprudhomme @github-code-quality
Potential fix for pull request finding 'Unused variable, import, func…
f9783dc 
…tion or class'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@fbeaudoincoveo fbeaudoincoveo fbeaudoincoveo approved these changes

@alexprudhomme alexprudhomme Awaiting requested review from alexprudhomme

@y-lakhdar y-lakhdar Awaiting requested review from y-lakhdar y-lakhdar was automatically assigned from coveo/dxui

@louis-bompart louis-bompart Awaiting requested review from louis-bompart

+1 more reviewer

@github-code-quality github-code-quality[bot] github-code-quality[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@alexprudhomme alexprudhomme

Copilot code review Copilot

Labels

area:atomic area:documentation area:e2e-tests area:insight chromatic Opt-out of Chromatic testing new-lit-component team:dev-writers team:dxui team:knowledge-ui-kit-reviewers team:search-qa

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@louis-bompart @fbeaudoincoveo @alexprudhomme