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

wip #76 begin work on ChatComponent #123

Merged
merged 8 commits into from
Mar 29, 2017
Merged

wip #76 begin work on ChatComponent #123

merged 8 commits into from
Mar 29, 2017

Conversation

simon-bourque
Copy link
Collaborator

Currently blocked until the study groups become available in the users object. For now we can merge what I have so far so we have something to show for sprint 4

@coveralls
Copy link

Coverage Status

Coverage increased (+1.2%) to 78.947% when pulling 56d8dd3 on chat_front into 149025d on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.2%) to 78.947% when pulling 4ded3e9 on chat_front into 149025d on master.

@pbgnz
Copy link
Contributor

pbgnz commented Mar 23, 2017

Phil's commit should allow the FE to get the study groups objects from the user

@simon-bourque
Copy link
Collaborator Author

This is not ready to be tested yet, We need a way to actually add other people to the study groups. I tried doing it manually in the database but I couldn't get it to work

@coveralls
Copy link

Coverage Status

Coverage increased (+3.03%) to 86.364% when pulling f61853d on chat_front into d1fc7c1 on master.

@Philippe229
Copy link
Collaborator

@simon-bourque this PR needs to be merged first because I made changes that should allow the person working on the study groups to add a study group to a user

@simon-bourque
Copy link
Collaborator Author

I was planning on discussing all this in the tutorial today because I am also blocking @samantha-kerr who needs the commits you made in this branch to implement the study group switching

@coveralls
Copy link

Coverage Status

Coverage increased (+3.03%) to 86.364% when pulling 99f2c93 on chat_front into d1fc7c1 on master.

Copy link
Collaborator

@Philippe229 Philippe229 left a comment

Choose a reason for hiding this comment

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

Front end code looks good. Tested and works as intended

@ONourry ONourry merged commit 9a33abb into master Mar 29, 2017
@MartinSpasov MartinSpasov deleted the chat_front branch April 2, 2017 22:09
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

5 participants