Skip to content

Conversation

@juanmitriatti
Copy link
Contributor

UDS-1946

Updated the logo alt and title in the brand header of the component-header and component-header-footer packages.

Description

Links

@juanmitriatti juanmitriatti requested a review from a team as a code owner March 24, 2025 17:57
Copy link
Contributor

@davidornelas11 davidornelas11 left a comment

Choose a reason for hiding this comment

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

Changes needed

className="vert"
src={asuLogo?.src ?? asuVertLogo}
alt={asuLogo?.alt ?? "Arizona State University"}
alt={asulogo?.alt ?? "Arizona State University logo"}
Copy link
Contributor

Choose a reason for hiding this comment

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

The asuLogo variable was renamed on accident to asulogo and that will end up erroring out here

className="vert"
src={asuLogo?.src ?? asuVertLogo}
alt={asuLogo?.alt ?? "Arizona State University"}
alt={asulogo?.alt ?? "Arizona State University logo"}
Copy link
Contributor

Choose a reason for hiding this comment

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

same on this file

@asu-jenkins-devops
Copy link
Collaborator

@davidornelas11 davidornelas11 merged commit 8126d47 into dev Mar 31, 2025
1 check passed
@davidornelas11 davidornelas11 deleted the UDS-1946 branch March 31, 2025 21:51
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.

4 participants