Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Website: Add favicon #816

Merged
merged 1 commit into from
Jan 5, 2025
Merged

Website: Add favicon #816

merged 1 commit into from
Jan 5, 2025

Conversation

andrew032011
Copy link
Collaborator

Summary

Add favicon to website.

@Bookie0 provided an .ico file.

Notion/Figma Link

Test Plan

Notes

Breaking Changes

@andrew032011 andrew032011 requested a review from a team as a code owner January 5, 2025 22:33
@dti-github-bot
Copy link
Member

[diff-counting] Significant lines: 0.

@andrew032011 andrew032011 changed the title Add favicon Website: Add favicon Jan 5, 2025
Bookie0
Bookie0 approved these changes Jan 5, 2025
View reviewed changes
Copy link
Collaborator

@Bookie0 Bookie0 left a comment

Choose a reason for hiding this comment

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

lets goooo

@andrew032011 andrew032011 merged commit d609077 into main Jan 5, 2025
17 checks passed
@andrew032011 andrew032011 deleted the axc/favicon branch January 5, 2025 22:45
@andrew032011 andrew032011 mentioned this pull request Jan 5, 2025
Merged
andrew032011 added a commit that referenced this pull request Jan 5, 2025
@andrew032011
Add slightly larger favicon (#817)
3cc1694 
### Summary <!-- Required -->
Add favicon to website that's larger than the one used here:
#816.

@Bookie0 provided an .ico file.

### Notion/Figma Link <!-- Optional -->

<!-- If the changes have associated Notion pages/Figma design(s), please
include the links here.-->

### Test Plan <!-- Required -->

<!-- Provide screenshots or point out the additional unit tests -->

### Notes <!-- Optional -->

<!--- List any important or subtle points, future considerations, or
other items of note. -->

### Breaking Changes <!-- Optional -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bookie0 Bookie0 Bookie0 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@andrew032011 @dti-github-bot @Bookie0