PR #70

12 votes · 12 up · 0 down

View on GitHub
12
Total Votes
+12
Upvotes
-0
Downvotes
+12-0

Comments(4)

▸
skridlevskyComment#70feat: Add a world chaos headline banner component.

@hpinsley, cool concept! The preview shows just the placeholder though - RSS fetch is hitting CORS since it's client-side:

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://rss.nytimes.com/services/xml/rss/nyt/World.xml. (Reason: CORS header 'Access-Control-Allow-Origin' missing).

Moving it to an API route would fix it.

▸
openchaos-bot[bot]Comment#70feat: Add a world chaos headline banner component.

🤖 OpenChaos Bot

Summary: This PR introduces a new "WorldChaos" component that displays a rotating headline from an RSS feed (defaulting to the NYT World feed). It also adds the 'rss-parser' dependency to consume RSS feeds.

Files changed: 4 (package.json, package-lock.json, src/app/page.tsx, src/components/WorldChaos.tsx)

Impact: Medium - Adds a new feature to the homepage, introduces a new dependency, and fetches external data.

openchaos-bot

▸
hpinsleyComment#70feat: Add a world chaos headline banner component.

@matthewmayer Haha yeah. Felt the breadth and scope of the headlines would lend itself to the ambiance and spirit of the site. ;-)

▸
matthewmayerComment#70feat: Add a world chaos headline banner component.

I was intrigued to see how you were explicitly going to find news about world chaos, only to find this is literally just the NY Times World News feed. Accurate.

All Activity(17)

▸
skridlevskyComment#70feat: Add a world chaos headline banner component.

@hpinsley, cool concept! The preview shows just the placeholder though - RSS fetch is hitting CORS since it's client-side:

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://rss.nytimes.com/services/xml/rss/nyt/World.xml. (Reason: CORS header 'Access-Control-Allow-Origin' missing).

Moving it to an API route would fix it.

▸
openchaos-bot[bot]Comment#70feat: Add a world chaos headline banner component.

🤖 OpenChaos Bot

Summary: This PR introduces a new "WorldChaos" component that displays a rotating headline from an RSS feed (defaulting to the NYT World feed). It also adds the 'rss-parser' dependency to consume RSS feeds.

Files changed: 4 (package.json, package-lock.json, src/app/page.tsx, src/components/WorldChaos.tsx)

Impact: Medium - Adds a new feature to the homepage, introduces a new dependency, and fetches external data.

openchaos-bot

▸
hpinsleyComment#70feat: Add a world chaos headline banner component.

@matthewmayer Haha yeah. Felt the breadth and scope of the headlines would lend itself to the ambiance and spirit of the site. ;-)

▸
matthewmayerComment#70feat: Add a world chaos headline banner component.

I was intrigued to see how you were explicitly going to find news about world chaos, only to find this is literally just the NY Times World News feed. Accurate.