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

Final changes to sprint 4 #66

Merged
merged 35 commits into from
Nov 1, 2018
Merged

Final changes to sprint 4 #66

merged 35 commits into from
Nov 1, 2018

Conversation

ribal-aladeeb
Copy link
Collaborator

Finished converting to react
Finished routes by refactoring them, fully implement likes and dislikes
Group pages has been added and we did integration of login/signup with JWT token
Added a Makefile to run TravisCI
Issues:
#63, #56, #46, #43, #42, #49, #47, #54

DPigeon (David) and others added 30 commits October 29, 2018 13:57
To see likes and dislikes from people
Testing
Just testing
Getting ready to start groups
When a user signups up he is redirected to login and when he logs in he
is redirected to dashboard.
Likes and dislikes are completed.
- Bug to fix: likes counter starts at -1, not 0.
- Enhanced posts.
Fixing conflicts
Fixing conflicts
Fixing conflicts
Fixing conflicts
Fixing conflicts
Fixing conflicts
Fixing conflicts
Fixing conflicts
Merging like-dislike into dev
- JWT is now decoded
- Name and email shows up on home page
- Routes added back to home like before
- Refactored the Routes Quickly
- JWT decodes in better way
DPigeon and others added 5 commits October 31, 2018 15:27
Testing TravisCI on frontend
Fixing the TravisCI error
- Fixed and removed some code in posts.
Need:
-Help with integrating to rest of site
-Review to make sure it is compatible with he rest of the site(Review by Marwan,David,Maria)
@ribal-aladeeb ribal-aladeeb changed the title Final iteration Final changes to sprint 4 Oct 31, 2018
@DPigeon DPigeon merged commit 22299fa into master Nov 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants