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

Add podcast feed URL status indicator #80

Merged
merged 5 commits into from
Jan 7, 2021

Conversation

alexdevero
Copy link
Contributor

This is based on issue #73.

This implementation creates new prop feedUrlStatus in poddycast-favorite_podcasts.json. This prop has value of 200 for working URL and 500 for broken URL.

When user adds new podcast to his favorites, the default value of feedUrlStatus is 200. Every time the app, or user, fetches new episodes this prop is updated. If podcast data are fetched successfully, feedUrlStatus is set to 200. If there is any error with the request feedUrlStatus is set to 500.

When user opens the "Favorites" view, an indicator is displayed based on the value of feedUrlStatus. This indicator is red border around the podcast thumbnail image (this might need change; there might be red-ish thumbnails). This indicator is created through CSS and new class on the thumbnail image.

Broken URL example:
p 1

Working URL example:
p2

@alexdevero alexdevero self-assigned this Dec 31, 2020
@MrChuckomo MrChuckomo linked an issue Dec 31, 2020 that may be closed by this pull request
@alexdevero
Copy link
Contributor Author

Small design improvement for podcast status indicator - new icon and info text on hover:
icon

@alexdevero
Copy link
Contributor Author

@MrChuckomo Have you look at this?

Copy link
Owner

@MrChuckomo MrChuckomo left a comment

Choose a reason for hiding this comment

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

Looks all good. 👍
It's a really nice new feature.

@alexdevero alexdevero merged commit 41fbb23 into master Jan 7, 2021
@alexdevero alexdevero deleted the add-saved-feed-url-status branch January 7, 2021 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option to test status of saved podcast URL
2 participants