Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

Added Starwars Dashboard with pagination #1

Merged
merged 1 commit into from
Dec 27, 2019

Conversation

xatuke
Copy link

@xatuke xatuke commented Dec 25, 2019

Changes:
Added a Dashboard which shows starwars data and you can paginate throught it.

Screenshots:
star1

DEMO

Copy link

@divyanshu-rawat divyanshu-rawat left a comment

Choose a reason for hiding this comment

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

I have suggested some changes to follow new syntax and enhance best practises feel free to make changes if you want 👍 .

src/App.css Outdated Show resolved Hide resolved
src/components/HeaderTabs.js Show resolved Hide resolved
src/components/MainPage.js Show resolved Hide resolved
src/components/sets/Films.js Outdated Show resolved Hide resolved
src/components/sets/People.js Outdated Show resolved Hide resolved
src/components/sets/Planets.js Outdated Show resolved Hide resolved
src/components/sets/Planets.js Outdated Show resolved Hide resolved
src/components/sets/Species.js Outdated Show resolved Hide resolved
src/components/sets/Starships.js Outdated Show resolved Hide resolved
src/components/sets/Vehicles.js Outdated Show resolved Hide resolved
@xatuke
Copy link
Author

xatuke commented Dec 26, 2019

@divyanshu-rawat Thanks for all the suggestions, this really helps me as I am new to React so I get to know better practices :) Will fix itall now 😄

@xatuke xatuke force-pushed the paginate-task branch 2 times, most recently from d98b7a4 to 6d7ef40 Compare December 26, 2019 14:17
Made requested changes :)


Made requested changes by Anukriti


Improved UX
@xatuke
Copy link
Author

xatuke commented Dec 29, 2019

let me know, if I should raise another PR as I see you have merged and reverted this one. @anukritijha

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants