Skip to content

Final topic #5

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
Open
rodrysauco wants to merge 53 commits into
base: master
Choose a base branch
from
Open

Final topic #5

rodrysauco wants to merge 53 commits into from

Conversation

@rodrysauco
Copy link
Owner

@rodrysauco rodrysauco commented Jul 16, 2018 •
edited
Loading

Working better than we expected

rodrysauco and others added 30 commits July 13, 2018 16:48
@rodrysauco
Initial Structure
7732430 
First commit
@matoliva
initial structure
336fd63
@matoliva
add axios dependency
b579d27
@matoliva
mock api
ef40331
@matoliva
update jobRow component
0ee3d44
@crpache
update JobList component
c7db6c3
@crpache
update Container/JobList css
19d02df
@crpache
add scroll overflow to JobList
a26b510
@crpache
add addFavHandler() method
dd011f8
@crpache
add removeFavHandler() method
1dd5a05
@crpache
update addFavHandler() to avoid adding fav more than once
269e424
@crpache
update jobrow.css
84d44a3
@crpache
add necesaries methods to save state on local storage
6f8724f
@crpache
update lifecycles
4c228bc
@crpache
update
3dae57b
@matoliva
add a jobDetails container
027281e
@matoliva
resolved merge conflict
0ed2616
@matoliva
update componentDidMount method
5295f84
@crpache
add list titles and fav counter
03209df
@rodrysauco
SearchBar working
b5c9bb1
@rodrysauco
AutoStash
48034b1
@crpache
update css
fa46000
@matoliva
add font awesome
43fc191
@crpache
fix conflicts
b52d692
@crpache
update search bar css and add results counter
08211c9
@crpache
fix local storage to avoid saving allJobs state
50bf47d
@matoliva
add showJobDetails method
de8f0bb
@crpache
update jobist css
89c9264
@crpache
fix merge conflicts
5c3a511
@rodrysauco
Changes
4915ffc 
Used proxy, and updated readme
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.

4 participants

@rodrysauco @matoliva @crpache