Skip to content

fix(knowledge): Replacing knowledge svgs with lighter versions#8930

Merged
preetriti1 merged 1 commit intomainfrom
priti/fiximages
Mar 17, 2026
Merged

fix(knowledge): Replacing knowledge svgs with lighter versions#8930
preetriti1 merged 1 commit intomainfrom
priti/fiximages

Conversation

@preetriti1
Copy link
Contributor

Commit Type

  • feature - New functionality
  • fix - Bug fix
  • refactor - Code restructuring without behavior change
  • perf - Performance improvement
  • docs - Documentation update
  • test - Test-related changes
  • chore - Maintenance/tooling

Risk Level

  • Low - Minor changes, limited scope
  • Medium - Moderate changes, some user impact
  • High - Major changes, significant user/system impact

What & Why

Updating knowledge svgs from 5 MB to < 10KB, this was causing big increase in bundle size hence breaking our build

Impact of Change

  • Users: No change
  • Developers: Lighter svgs make the bundles more consumable in portal side
  • System: Faster load time of these static images.

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in:

Contributors

@bonicaayala

Screenshots/Videos

NA

@preetriti1 preetriti1 added the risk:low Low risk change with minimal impact label Mar 17, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@preetriti1 preetriti1 enabled auto-merge (squash) March 17, 2026 09:48
@github-actions
Copy link

❌ PR Validation Error

An error occurred while validating your PR. Please try again later or contact the maintainers.

Error: API request failed with status 504

1 similar comment
@github-actions
Copy link

❌ PR Validation Error

An error occurred while validating your PR. Please try again later or contact the maintainers.

Error: API request failed with status 504

@github-actions
Copy link

📊 Coverage Check

No source files changed in this PR.

@preetriti1 preetriti1 merged commit a1aa622 into main Mar 17, 2026
16 of 19 checks passed
@preetriti1 preetriti1 deleted the priti/fiximages branch March 17, 2026 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:low Low risk change with minimal impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants