Skip to content

20260309 ada chart fixes#489

Merged
rseward merged 11 commits intomasterfrom
20260309-ada-chart-fixes
Mar 9, 2026
Merged

20260309 ada chart fixes#489
rseward merged 11 commits intomasterfrom
20260309-ada-chart-fixes

Conversation

@som-sewardr1
Copy link

Ticket Link #fill_in

Remove if not applicable

Description

Added discovery-ui chart support for LINK_COLOR (link_color) to allow overriding the brand default link color. (Largely for ADA compliance reasons.)

Reminders

  • Did you up the relevant chart version numbers? (If appropriate)
    • If you up a chart version within urban-os, have you also upped the urban-os chart version itself?
    • If charts within the urban-os chart (andi, etc) have been updated, have you run helm dependency updateh in /charts/urban-os and commited the Chart.lock file?
  • Does helm template . -f values.yaml pass? (Checks for default values provided in chart and catches other errors)
  • Do you have git hooks installed? (See README.md to install)
  • If global values were altered, are they included as chart default values?
    • Are they also specified in the urbanos chart values file?
  • If references to external charts were added:
    • Was the github release action updated to helm update {new_thing} it's dependencies?
    • Was the deploy repo -u flag updated to helm update {new_thing} to ensure it's not left out of deployments?

@som-sewardr1 som-sewardr1 requested a review from rseward March 9, 2026 19:29
Copy link
Contributor

@rseward rseward left a comment

Choose a reason for hiding this comment

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

Code changes appear fit for purpose.

@rseward rseward merged commit 461eca1 into master Mar 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants