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.

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

Update website content #1668

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Update website content #1668

wants to merge 7 commits into from

Conversation

rileyhgrant
Copy link
Contributor

@rileyhgrant rileyhgrant commented Jan 6, 2025

Updates website content

  • Adds 'human' to about page
  • Removes member of SAB who is no longer member
  • Adds 'tm' to gnomAD where relevant
  • Moves and edits Open Science Policy
  • Adds gnomAD videos FAQ
  • Adds "Federated gnomAD" page
  • Rework citations page (re-organize, add new papers, use component for reusable citation)
  • Updates the Team page - moves Steve to Alumni :(

@rileyhgrant rileyhgrant self-assigned this Jan 6, 2025
@rileyhgrant rileyhgrant force-pushed the jan-2025-content-updates branch 3 times, most recently from d62cb4a to 8e45535 Compare January 8, 2025 15:16
Copy link
Contributor

@phildarnowsky-broad phildarnowsky-broad left a comment

Choose a reason for hiding this comment

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

There's a couple of things I'm dubious about, I'm guessing you had good reasons for most or all of them but I'd like to understand the reasoning too.

browser/about/about.md Show resolved Hide resolved
browser/about/policies/terms.md Outdated Show resolved Hide resolved
browser/src/FederationPage.tsx Outdated Show resolved Hide resolved
browser/src/FederationPage.tsx Show resolved Hide resolved
browser/src/FederationPage.tsx Show resolved Hide resolved
browser/src/FederationPage.tsx Outdated Show resolved Hide resolved
@rileyhgrant
Copy link
Contributor Author

Heya @phildarnowsky-broad

Added two fixup commits

  • Removed the one unneccesary @ts-expect-error that was called out
  • Added alt text for the logos -- thanks for the catch!

@rileyhgrant rileyhgrant force-pushed the jan-2025-content-updates branch 3 times, most recently from 676f6a0 to c694d68 Compare January 9, 2025 22:46
@rileyhgrant rileyhgrant force-pushed the jan-2025-content-updates branch from c694d68 to f5262cf Compare January 9, 2025 22:47
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