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

font-serif Taviraj purge and Terms of Use updates #3902

Merged
merged 7 commits into from
Jan 9, 2025

Conversation

kccrtv
Copy link
Collaborator

@kccrtv kccrtv commented Jan 8, 2025

Description and Motivation

  • rm font-serif taviraj around the app except for landing page hero
  • updates hero text size on md and lg breakpoints
  • updates terms of use UI
  • updates terms of use content
  • rm unused news components that were previously using taviraj but not needed from latest refactor:
    • ArticleFilters & PinnedArticles

Has this been tested? How?

updated internal_routes test

Screenshots (if appropriate)

aa-methodology
tou
hero

Types of changes

  • Refactor / chore

New frontend preview link is below in the Netlify comment 😎

Copy link

netlify bot commented Jan 8, 2025

Deploy Preview for health-equity-tracker ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 913eb66
🔍 Latest deploy log https://app.netlify.com/sites/health-equity-tracker/deploys/67802797fef75a000859e1fe
😎 Deploy Preview https://deploy-preview-3902--health-equity-tracker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@kccrtv kccrtv changed the title remove taviraj, update hero text size, update TOU content, rm unused … font-serif Taviraj purge and Terms of Use updates Jan 8, 2025
@kccrtv kccrtv marked this pull request as ready for review January 8, 2025 16:50
benhammondmusic and others added 5 commits January 9, 2025 13:46
SatcherInstitute#3908)

By switching the codebase's `console.log` statements into more specific
ones like `.info()` `.error()` `.warn()`, we provide better developer
feedback, and we also make it easier to find and remove any temporary
console.log statements that have been added and overlooked during the
local dev process
@kccrtv kccrtv merged commit 266e727 into SatcherInstitute:main Jan 9, 2025
11 checks passed
@kccrtv kccrtv deleted the taviraj branch January 9, 2025 19: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.

2 participants