Skip to content

Conversation

dchyun
Copy link
Contributor

@dchyun dchyun commented Jul 30, 2025

📌 Summary

If merged this PR would update an imported type for COLORS to its new type name, which was changed recently in #3063 .

It also fixes a couple incorrect path names in the acceptance test comments of the website icons.


💬 Please consider using conventional comments when reviewing this PR.

📋 PCI review checklist
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've worked with GRC to document the impact of any changes to security controls.
    Examples of changes to controls include access controls, encryption, logging, etc.
  • If applicable, I've worked with GRC to ensure compliance due to a significant change to the in-scope PCI environment.
    Examples include changes to operating systems, ports, protocols, services, cryptography-related components, PII processing code, etc.

Copy link

vercel bot commented Jul 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview Jul 30, 2025 4:57pm
hds-website ✅ Ready (Inspect) Visit Preview Jul 30, 2025 4:57pm

@hashibot-hds hashibot-hds added the docs-website Content updates to the documentation website label Jul 30, 2025
@dchyun dchyun marked this pull request as ready for review July 30, 2025 16:55
@dchyun dchyun requested review from a team as code owners July 30, 2025 16:55
@dchyun dchyun requested review from didoo and shleewhite July 30, 2025 16:55
@dchyun dchyun merged commit 2c1715c into main Jul 30, 2025
12 checks passed
@dchyun dchyun deleted the dchyun/website-type-name-fixes branch July 30, 2025 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-website Content updates to the documentation website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants