Add Search-by-Name Feature for RSS Feed Input#75
Open
raza-khan0108 wants to merge 1 commit intoamcquade:masterfrom
Open
Add Search-by-Name Feature for RSS Feed Input#75raza-khan0108 wants to merge 1 commit intoamcquade:masterfrom
raza-khan0108 wants to merge 1 commit intoamcquade:masterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a “search by podcast name” path to the RSS feed input flow by querying the iTunes Search API when the submitted input doesn’t look like a URL, and presenting selectable matches in a dialog.
Changes:
- Updated the RSS input placeholder/ARIA label to indicate URL-or-name entry.
- Refactored feed loading into a
fetchRSS(url)helper and added iTunes Search API lookup for name-based input. - Added a MUI dialog listing matching podcasts (with artwork) so a user can select a feed to load.
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| src/components/UserForm.jsx | Updates user-facing input text for URL-or-name entry; continues to submit via getFeed. |
| src/App.js | Implements name search (iTunes API), refactors RSS fetching, and adds a selectable results dialog UI. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| setFetched({ | ||
| episodes: feed.items, | ||
| program_title: feed.title, | ||
| program_image: feed.image.url, |
| const getFeed = async (event) => { | ||
| if (event && event.preventDefault != null) { | ||
| event.preventDefault(); | ||
| } |
| if (data.results && data.results.length > 0) { | ||
| const exactMatch = data.results.find(p => p.collectionName.toLowerCase() === input_val.toLowerCase()); | ||
| if (exactMatch && exactMatch.feedUrl) { | ||
| setFetching(false); |
Comment on lines
+90
to
+91
| setSearchResults(data.results.filter(p => p.feedUrl)); | ||
| setSearchDialogOpen(true); |
Comment on lines
22
to
+27
| <Input | ||
| placeholder="Enter your RSS Feed here..." | ||
| placeholder="Enter an RSS Feed URL or search by Name..." | ||
| type="text" | ||
| name="feed_url" | ||
| onChange={handleSearchChange} | ||
| aria-label="Enter your RSS Feed here..." | ||
| aria-label="Enter an RSS Feed URL or search by Name..." |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Enhanced the RSS feed input to support both direct URL entry and name-based podcast search, so users can discover feeds without knowing the exact RSS URL.
Changes
src/components/UserForm.jsx
aria-labelto"Enter an RSS Feed URL or search by Name..."handleSearchChangehandler to detect URL vs. name inputsearchResultsandsearchDialogOpenstate variablesrenderSearchDialog()to display matching results in a MUI dialogsrc/App.js
List,ListItem, etc.)How It Works
Future Work