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

cleanup static images #2712

Closed
ttarsi opened this issue Dec 17, 2024 · 0 comments · Fixed by #2729
Closed

cleanup static images #2712

ttarsi opened this issue Dec 17, 2024 · 0 comments · Fixed by #2729

Comments

@ttarsi
Copy link
Contributor

ttarsi commented Dec 17, 2024

Problem to Solve

  1. There are still some images with the old brand kit in docs/website/static/img. It is unclear if they’re actually used or not.
  2. The readme banner also uses old branding, let’s replace it.
  3. While we’re at it we could deprecate the static directory, since we’re now hosting many images at docs.omni.network. Always love removing dirs. This will require update Eigenlayer AVS data on chain.

Proposed Solution

  • remove any images that are unused
  • replace any instances of the old branding that are still used with new branding
  • replace readme banner
  • update eigen avs metadata to use something hosted at docs.omni.network/img/, and remove the static repo
@ttarsi ttarsi mentioned this issue Dec 17, 2024
Merged
ttarsi added a commit that referenced this issue Dec 18, 2024
@ttarsi
chore(docs): cleanup unused static docs files (#2715)
30f095a 
1. Clean up unused static files in docs. When the preview was showing
our old branding yesterday, it was difficult to know which files were
being used. This removes files that are unused so that it's easier to
know what is live.
2. Moves `static-avs-metadata.json` to be hosted on the docs site, so
that we can deprecate the `static` directory (less is more, want to make
the monorepo less confusing). Still need to make an onchain transaction
so our AVS contract still displays correctly on the eigen frontend
(updates where to pull metadata from) but we can merge this and then
remove the static dir later.

issue: #2712
@ttarsi ttarsi mentioned this issue Dec 19, 2024
Merged
ttarsi added a commit that referenced this issue Dec 20, 2024
@ttarsi
chore(docs): use brand colors in badges (#2721)
b774c76 
uses omni brand colors. #consistency

issue: #2712
@ttarsi ttarsi mentioned this issue Dec 20, 2024
Merged
@ttarsi ttarsi closed this as completed in 1d467a4 Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore(*): remove static dir omni-network/omni
1 participant
@ttarsi