Skip to content

Conversation

@psterpu-c
Copy link
Contributor

πŸ“ Description

πŸ”— Related Issue

Closes #(issue number)

πŸ§ͺ Type of Change

  • πŸ› Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • πŸ’₯ Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • πŸ“š Documentation update
  • 🎨 Style/design update
  • πŸ”§ Build/CI update
  • ♻️ Refactoring (no functional changes)
  • πŸ§ͺ Tests only

πŸ§ͺ Testing

  • Unit tests pass (yarn test)
  • Storybook builds successfully (yarn storybook)
  • Manual testing completed
  • Cross-browser testing (if applicable)

Test Instructions:

β™Ώ Accessibility

  • Follows WCAG 2.1 AA guidelines
  • Keyboard navigation tested
  • Screen reader tested
  • Color contrast verified
  • Focus indicators present
  • ARIA attributes added where needed

πŸ“± Responsive Design

  • Mobile responsive
  • Tablet responsive
  • Desktop responsive
  • All breakpoints tested

πŸ“š Documentation

  • Storybook story added/updated
  • Component props documented
  • Usage examples provided
  • README updated (if needed)
  • TypeScript types exported

πŸ”„ Breaking Changes

  • This PR introduces breaking changes
  • Migration guide provided (if breaking changes)
  • Version bump required

πŸ“‹ Checklist

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Code is commented where necessary
  • No console.log statements left in code
  • Build passes locally
  • Tests added for new functionality
  • Existing tests still pass
  • No TypeScript errors
  • No accessibility violations
  • PR title follows conventional commit format

πŸ“Έ Screenshots

Before

After

πŸ” Additional Notes


For Maintainers:

  • Ready for review
  • Requires design review
  • Requires accessibility review
  • Ready to merge

@psterpu-c psterpu-c requested a review from a team as a code owner October 3, 2025 06:39
Signed-off-by: Paul Sterpu <psterpu@cisco.com>
Signed-off-by: Paul Sterpu <psterpu@cisco.com>
@psterpu-c psterpu-c merged commit b57f70f into main Oct 3, 2025
7 checks passed
@psterpu-c psterpu-c deleted the spider-chart branch October 3, 2025 06:48
@rafaelsilva29
Copy link
Member

πŸŽ‰ This PR is included in version 1.2.1 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants