Skip to content

Conversation

@erikenrique
Copy link
Collaborator

@erikenrique erikenrique commented Mar 29, 2025

🤔 Describe your changes

Integrated TheNewsAPI to have news headlines we can display on our feed website

Issue ticket number and link (also any related tickets)

#120
(Ticket #120 )

Type of update

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • UI fix or UI update (be sure to provide a screenshot or .gif if this is checked)
  • This change requires a documentation update
  • 🔥Hot fix🔥 (unscheduled fix, be sure to create a ticket for this)
  • Update to documentation

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • I have ensured that this PR contains no known bugs

Images or .gifs (this section is to display any UI updates or code snippets that can enhance our understanding of your changes)

image

Next Steps

  • make sure that Feed.jsx is fine - i noticed Feed.jsx wasn't included in my code after pulling from the most recent PR. So i copied and pasted from whats currently on the github (in the Feed component folder). Should be the exact same as what's on github)
  • make sure that .env file is updated to use our API key for TheNewsApi
  • make news just be tech specific (refer to documentation here) https://www.thenewsapi.com/documentation

Copy link
Collaborator

@ZariaHallager ZariaHallager left a comment

Choose a reason for hiding this comment

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

#125

Integrating the news api and making sure it is functional and does not break the code base.

@ZariaHallager ZariaHallager merged commit 723c3e4 into main Mar 29, 2025
1 check failed
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