Skip to content

refactor: tag color handling in announcement components#445

Open
julia-rabello wants to merge 1 commit intomainfrom
refactor/announcement-tag-color-handling
Open

refactor: tag color handling in announcement components#445
julia-rabello wants to merge 1 commit intomainfrom
refactor/announcement-tag-color-handling

Conversation

@julia-rabello
Copy link
Copy Markdown
Contributor

@julia-rabello julia-rabello commented Mar 31, 2026

What is the purpose of this pull request?

EDU-17952

  • Replaced hardcoded tag color logic with a centralized function getTagColorByLocalizedName for better maintainability.
  • Updated AnnouncementCard and AnnouncementsDropdown components to utilize the new function for determining tag colors.
  • Removed redundant type color mappings from the components, enhancing code clarity.

What problem is this solving?

How should this be manually tested?

Screenshots or example usage

Types of changes

  • 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 change)
  • Requires change to documentation, which has been updated accordingly.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 31, 2026

Deploy Preview for leafy-mooncake-7c2e5e ready!

Name Link
🔨 Latest commit 8c32158
🔍 Latest deploy log https://app.netlify.com/projects/leafy-mooncake-7c2e5e/deploys/69cc033140990d0009ec8a6d
😎 Deploy Preview https://deploy-preview-445--leafy-mooncake-7c2e5e.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@julia-rabello julia-rabello self-assigned this Mar 31, 2026
Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: ['release-no', 'release-auto', 'release-patch', 'release-minor', 'release-major']

@julia-rabello julia-rabello added the release-auto Automatic version bump label Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-auto Automatic version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant