Skip to content

Added Favorites List #202

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 31 commits into from
Closed

Added Favorites List #202

wants to merge 31 commits into from

Conversation

austintestut
Copy link

No description provided.

@austintestut
initial separation of concerns of API and React app
1e3e47b
@austintestut
locked graphQL server to port 8080
27944b4
@austintestut
revert
f1de34e
@austintestut
add dependencies
8e4f744
@austintestut
added Apollo functionality, rendered names in character container
6295afa 
component
@austintestut
querying all necessary data (all at once currently)
882d777
@austintestut
Merge pull request #1 from austintestut/queryData
3d0cb9f 
querying all necessary data (all at once currently)
@austintestut
added star images for favoriting characters
eb298fe
@austintestut
moved assets folder for import compatibility
062376b
@austintestut
separated each character into their own new component
5136968
@austintestut
added (currently only asthetic) toggle functionality to favorite-stars
4c0aa65
@austintestut
clicking a star adds character to local storage list
b47b4a9
@austintestut
star fill persists for favorited characters upon page refresh
b411cc6
@austintestut
Merge pull request #2 from austintestut/favoriting
6a16268 
Favoriting
@austintestut
refactored to make gql call in App component and pass down info,
3206d37 
allowing search bar to stay in header and have access to data
@austintestut
created search bar and header components
72422a3
@austintestut
passed searchCharacters function to search bar
7c860f6
@austintestut
searching names works upon typing in search bar
74082d6
@austintestut
Merge pull request #3 from austintestut/search
f7984f4 
Search
@austintestut
fixed bug where a favorited character would stay favorited after first
4063bb7 
click after a refresh
@austintestut
Merge pull request #4 from austintestut/favoritingBug
929d806 
fixed bug where a favorited character would stay favorited after first click after a refresh
@austintestut
fixed bug where submitting search field caused page refresh
6ae7e33
@austintestut
Merge pull request #5 from austintestut/search
b40ee96 
fixed bug where submitting search field caused page refresh
@austintestut
implemented client-side sorting by character name
edd51b0
@austintestut
Merge pull request #6 from austintestut/sorting
6ff570f 
implemented client-side sorting by character name
@austintestut
installed styled-components dependency
2a83834
@austintestut
basic styling of header, added border, centered elements
b7ffe44
@austintestut
basic styling and coloring of search button
a5fc49d
@austintestut
basic styling of character cards
30425f5
@austintestut
arranged cards so first 10 show
c6929cd
@linux-foundation-easycla
Copy link

CLA Not Signed

  • ❌ - Austin Testut The commit (1e3e47b ,27944b4e281896e23390946ab640c082864099c2 ,f1de34eb551ed11a2d4f5fa4db808052214cce7a ,8e4f7441f9a99ddbb04a1868505e9ab9602f9aec ,6295afa6ee1a947a1fc8bb1147ddf37ad43df761 ,882d7778119a75ea1f89bfe3d556af27f0c6bfce ,3d0cb9ff8bbe197fcf46565074d684f87cb5b573 ,eb298fe2609a2449a0083299909699408b698629 ,062376b4cc0862c68f6ea05a1a2c004714534758 ,51369684ead8f29b3a8d5da6ad6fec190515d8f2 ,4c0aa65b2f9b306d5c46d22f42f60a7fd7371b92 ,b47b4a9b34e1fe6b9bd4bfa37d5dad12b4c9e310 ,b411cc669af77b046a90928e4880464dda122013 ,6a162682613b69f3a1c758cb2da8d7adb9e8cbdb ,3206d37bcd05cee0f2b6f7642ba13319cc4ed62f ,72422a37a86e4c95bb148e4ca1457109f29e7768 ,7c860f6bc81d2ce3a9ade1e5c12d79bc1d8d3f27 ,74082d6a9d6c733b3712fbc8e6a6446742edcf13 ,f7984f4d50a88e7020df36db7d0b10d101104ec3 ,4063bb7cec5f45af8ea784a26495664d92b176bc ,929d806712bf828d1a603ac1ba0ce7de74a1526a ,6ae7e3333b333f659e0600883d25dd04f4ea2ed1 ,b40ee96dd04a7164ed1b2ba258a10a6eee6a7aad ,edd51b0eb35b67670486d929bd03a86c7710a338 ,6ff570f09f970872918ba46b9baeab6e9f9a5946 ,2a83834540d4ade4db3605115e5b64cef26e5b74 ,b7ffe447d522573a57cc3438ecb1c3a9ecb87b39 ,a5fc49dd59d2f4a49337467f89c19da31126b038 ,30425f55a5cc147098b30def80007cb99c35952b ,c6929cd8080cb4483d076b8c3c329679da871908 ,9dc93e009bc979cd1a5da0bf817687973a4a7413) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

@austintestut austintestut deleted the styling branch June 18, 2021 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@austintestut