Skip to content
This repository has been archived by the owner on Aug 21, 2018. It is now read-only.

[#1047] Community member enrolled on course (connects #1047) #1049

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

[#1047] Community member enrolled on course (connects #1047) #1049

wants to merge 2 commits into from

Conversation

razu9861
Copy link
Contributor

screen shot 2017-10-12 at 11 31 07 am
screen shot 2017-10-12 at 11 31 29 am

lmmrssa
lmmrssa reviewed Oct 12, 2017
View reviewed changes
app/MyApp/app/views/CourseMembers.js Outdated
@@ -63,7 +63,7 @@ $(function () {
var na = currentConfig.rows[0].doc.nationName.substring(3,5);
$('.courseEditStep').empty();
$('.courseEditStep').append('<h3>'+App.languageDict.attributes.Course_Members+ ' | ' + courseModel.get('name') + '</h3>')
var viewtext = '<table id = "Table1" class="btable btable-striped"><th>'+App.languageDict.attributes.Photo+'</th><th>'+App.languageDict.attributes.Name+'</th><th>'+App.languageDict.attributes.Roles+'</th><th colspan=2>'+App.languageDict.attributes.Actions+'</th>'
var viewtext = '<table id = "Table1" class="btable btable-striped"><th>'+App.languageDict.attributes.Photo+'</th><th>'+App.languageDict.attributes.Name+'</th><th>'+App.languageDict.attributes.Roles+'</th><th>'+App.languageDict.attributes.Community+'</th><th colspan="2">'+App.languageDict.attributes.Actions+'</th>'
Copy link
Member

Choose a reason for hiding this comment

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

It will show community name for single community list as well. It should only show for 'All'.

@lmmrssa
Copy link
Member

lmmrssa commented Oct 27, 2017

fix the conflict

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lmmrssa lmmrssa lmmrssa left review comments

@nirojdyola nirojdyola nirojdyola approved these changes

@Rupesh87 Rupesh87 Rupesh87 approved these changes

@Pr0chin Pr0chin Pr0chin approved these changes

@dogi dogi Awaiting requested review from dogi

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@razu9861 @lmmrssa @nirojdyola @Rupesh87 @Pr0chin