8000 Admin - Stream Comments by yj-soh · Pull Request #67 · nus-mtp/worldscope · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Admin - Stream Comments #67

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
Merged
merged 16 commits into from
Mar 15, 2016
Merged

Admin - Stream Comments #67

merged 16 commits into from
Mar 15, 2016

Conversation

yj-soh
Copy link
Contributor
@yj-soh yj-soh commented Mar 15, 2016
  • Display comments on stream page
    • Modifications to SocketAdapter for admins
  • Small changes
    • Hide end date on stream page if null
    • Fix dates to have leading zeros
    • Slight refactoring

@ndt93
Copy link
Contributor
ndt93 commented Mar 15, 2016

Good job 👍 Can merge.

yj-soh added a commit that referenced this pull request Mar 15, 2016
@yj-soh
Merge pull request #67 from nus-mtp/admin-stream-comments
5d2edb4 
Admin - Stream Comments
@yj-soh yj-soh merged commit 5d2edb4 into master Mar 15, 2016
@sharminelow sharminelow deleted the admin-stream-comments branch March 22, 2016 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ndt93 ndt93

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yj-soh @ndt93
0