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

Updated broken urls, moved one entry to the first table #207

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rafael-java
Copy link
Contributor

Hopefully I did well!

@@ -112,9 +113,9 @@ _(scroll right on table to see all 5 columns)_
| Headspace | [meditofoundation.org](https://meditofoundation.org/medito-app) | [GitHub](https://github.com/meditohq/medito-app) | Flutter | ![GitHub Repo stars](https://flat.badgen.net/github/stars/meditohq/medito-app) |
| Hacker News | [hackernews-redesign.netlify](https://hackernews-redesign.netlify.app/) | [GitHub](https://github.com/RocktimSaikia/hacker-news-redesign) | React | ![GitHub Repo stars](https://flat.badgen.net/github/stars/RocktimSaikia/hacker-news-redesign) |
| Hacker News | [news.python](https://news.python.sc/) | [GitHub](https://github.com/sebst/pythonic-news) | Python, Django | ![GitHub Repo stars](https://flat.badgen.net/github/stars/sebst/pythonic-news) |
| Hacker News | [rm-hackernews.netlify.app](https://rm-hackernews.netlify.app) | [GitHub](https://github.com/rajat-mehra05/hacker-news-clone) | Reactjs | ![GitHub Repo stars](https://flat.badgen.net/github/stars/rajat-mehra05/hacker-news-clone) |
| Hacker News | [github.com (screenshot)](https://github.com/rajat-mehra05/hackerpedia/blob/master/hnc.png) | [GitHub](https://github.com/rajat-mehra05/hackerpedia) | Reactjs | ![GitHub Repo stars](https://flat.badgen.net/github/stars/rajat-mehra05/hacker-news-clone) |
Copy link
Contributor

Choose a reason for hiding this comment

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

add the demo url instead of just the image https://hackerpedia.vercel.app/

| Hacker News | [hackernews-jaywhen.vercel.app](https://hackernews-jaywhen.vercel.app/) | [GitHub](https://github.com/jaywhen/hackernews) | Next.js | ![GitHub Repo stars](https://flat.badgen.net/github/stars/jaywhen/hackernews) |
| IMDB | [imdb-react-clone.herokuapp](https://imdb-react-clone.herokuapp.com/) | [GitHub](https://github.com/djohal/imdb-replica-react) | React, Redux, Firebase | ![GitHub Repo stars](https://flat.badgen.net/github/stars/djohal/imdb-replica-react) |
| IMDB | [github.com (screenshot)](https://github.com/djohal/imdb-replica-react) | [GitHub](https://github.com/djohal/imdb-replica-react) | React, Redux, Firebase | ![GitHub Repo stars](https://flat.badgen.net/github/stars/djohal/imdb-replica-react) |
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add the correct screenshot URL
Demo URL is not working but there a GIF which would much appropriate instead of the repo URL in the demo
https://github.com/djohal/imdb-replica-react/blob/master/imdb-clone-demo-min.gif

| Netflix | [github.com (screenshot)](https://github.com/karlhadwen/netflix/blob/master/netflix-preview.png) | [GitHub](https://github.com/karlhadwen/netflix) | React, Firebase | ![GitHub Repo stars](https://flat.badgen.net/github/stars/karlhadwen/netflix) |
| Netflix | [spaceflix.herokuapp](http://spaceflix.herokuapp.com/#/) | [GitHub](https://github.com/ehecker/spaceflix) | Ruby, React, Redux, PostgreSQL, AWS | ![GitHub Repo stars](https://flat.badgen.net/github/stars/ehecker/spaceflix) |
| Netflix | [azazel5.github.io](https://azazel5.github.io/NetflixClone/) <br/> [github.com (gif)](https://github.com/Azazel5/NetflixClone/blob/master/flixdemo.gif) | [GitHub](https://github.com/Azazel5/NetflixClone) | React, Redux | ![GitHub Repo stars](https://flat.badgen.net/github/stars/Azazel5/NetflixClone) |
| Netflix | [netflix-clone-dd230.web.app](https://netflix-clone-dd230.web.app/)[github.com (gif)](https://github.com/abodmicheal/React-projects/blob/master/gifs/20200915_142422.gif) | [GitHub](https://github.com/abodmicheal/React-projects/tree/master/Netflix%20UI%20Web%20App) | React, Firebase | ![GitHub Repo stars](https://flat.badgen.net/github/stars/abodmicheal/React-projects) | |
Copy link
Contributor

Choose a reason for hiding this comment

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

This project doesn't use Firebase except for hosting, Not sure if that should be included or not.
https://github.com/abodmicheal/React-projects/tree/master/Netflix%20UI%20Web%20App

@iamrishupatel
Copy link
Contributor

@rafael-java Please make the required changes and resolve merge conflicts

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.

None yet

3 participants