Skip to content

[BUG] Fix Instructor Name Schema #161

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

Closed
DerekTBrown opened this issue Jul 22, 2016 · 2 comments
Closed

[BUG] Fix Instructor Name Schema #161

DerekTBrown opened this issue Jul 22, 2016 · 2 comments
Assignees
Milestone

Comments

@DerekTBrown
Copy link
Member

DerekTBrown commented Jul 22, 2016

We need to modify the schema for courses to have instructors saved as a tuple of name and id. This then needs to be modified wherever we pull the instructor id and name (particularly the explore and search views).

@DerekTBrown DerekTBrown changed the title [REFACTOR [REFACTOR] Fix Instructor Name Schema Jul 22, 2016
@DerekTBrown DerekTBrown added this to the alpha.1 milestone Jul 22, 2016
@DerekTBrown DerekTBrown changed the title [REFACTOR] Fix Instructor Name Schema [BUG] Fix Instructor Name Schema Jul 22, 2016
@sandershihacker
Copy link

Already done in the Simple Schema, still need to fix in pages.

@DerekTBrown
Copy link
Member Author

Yes.. in particular the explore page

DerekTBrown pushed a commit that referenced this issue Jul 25, 2016
* Updated Schemas

* Load Screen

* Loading Screen

* publish subscribe and dynamic explore page

* deleted swap file

* publish subscribe

* template fixes

* added start for reactive publication

* template issue

* .

* .

* course_records publish

* course_records reactively publish

* revert meteor update

* publish subscribe and lineheight for searchpage icons

* fixed indentation and removed test file

* indentation

* indentation

* Accounts page and fixed random erro

* toolbar for mobile devices

* toolbar

* account.html fixes

* Toolbar made responsive

* Responsive toolbar fixes

* created userlist component

* course_records course_id validation

* indentation

* markdown

* import changes

* validation addition

* Forms Module Deprecated Warning. Issue #110

* indentation

* Use routerLink

* api changes (#126)

* issue 32 fixed

* removed swp

* fixed nconf issues

* issue 82, 84 fixes

* fixed #82

* isse #77

* fixed tests

* fixed files

* fixed tests master

* fixed comment

* moved validator

* removed duplicate file

* fixed tests, removed duplicate underscore, publish

* changes

* fixed methods.ts import

* fixed issues w methods

* fixes

* minor fixes

* A

* api changes

* fixed api

* comments

* minor changes

* minor fixes

* method implementations (#127)

* issue 32 fixed

* removed swp

* fixed nconf issues

* issue 82, 84 fixes

* fixed #82

* isse #77

* fixed tests

* fixed files

* fixed tests master

* fixed comment

* moved validator

* removed duplicate file

* fixed tests, removed duplicate underscore, publish

* changes

* fixed methods.ts import

* fixed issues w methods

* fixes

* minor fixes

* A

* api changes

* fixed api

* comments

* minor changes

* minor fixes

* method implementations

* course pages now uses Child Routing

* deleted LabData class

* school name

* search function

* icon size and #136

* dashboard pull data from database

* fix inappropriate file naming for gradelist and lablist

* more templating issues

* .

* .

* markdown default

* lablist and gradelist

* template changes

* explore page changes

* delete instructor page

* user schema

* indentation

* indentation

* explore publish and taskview sidenav

* explore view links to course/courseId

* taskview sidenav

* taskview

* taskview sidenav changes

* Extend terminal on small width

* .

* .

* Added UserList Component in Courses

* Edit Course

* .

* course dashboard get courseid from url

* delete userlist

* input

* delete userlist

* publish for global admin

* Fixed router params

* Issue 161 fixed: instructor field in courses

* .

* fix nextButton

* .
DerekTBrown pushed a commit that referenced this issue Jan 1, 2017
Issue #171

Issue #168

Issue 160: CourseList

Removed LabView

Enrollable field for buttons on explore pages

TaskView -> LabView

Fixed Router Params, Issue #161 (#174)

* Updated Schemas

* Load Screen

* Loading Screen

* publish subscribe and dynamic explore page

* deleted swap file

* publish subscribe

* template fixes

* added start for reactive publication

* template issue

* .

* .

* course_records publish

* course_records reactively publish

* revert meteor update

* publish subscribe and lineheight for searchpage icons

* fixed indentation and removed test file

* indentation

* indentation

* Accounts page and fixed random erro

* toolbar for mobile devices

* toolbar

* account.html fixes

* Toolbar made responsive

* Responsive toolbar fixes

* created userlist component

* course_records course_id validation

* indentation

* markdown

* import changes

* validation addition

* Forms Module Deprecated Warning. Issue #110

* indentation

* Use routerLink

* api changes (#126)

* issue 32 fixed

* removed swp

* fixed nconf issues

* issue 82, 84 fixes

* fixed #82

* isse #77

* fixed tests

* fixed files

* fixed tests master

* fixed comment

* moved validator

* removed duplicate file

* fixed tests, removed duplicate underscore, publish

* changes

* fixed methods.ts import

* fixed issues w methods

* fixes

* minor fixes

* A

* api changes

* fixed api

* comments

* minor changes

* minor fixes

* method implementations (#127)

* issue 32 fixed

* removed swp

* fixed nconf issues

* issue 82, 84 fixes

* fixed #82

* isse #77

* fixed tests

* fixed files

* fixed tests master

* fixed comment

* moved validator

* removed duplicate file

* fixed tests, removed duplicate underscore, publish

* changes

* fixed methods.ts import

* fixed issues w methods

* fixes

* minor fixes

* A

* api changes

* fixed api

* comments

* minor changes

* minor fixes

* method implementations

* course pages now uses Child Routing

* deleted LabData class

* school name

* search function

* icon size and #136

* dashboard pull data from database

* fix inappropriate file naming for gradelist and lablist

* more templating issues

* .

* .

* markdown default

* lablist and gradelist

* template changes

* explore page changes

* delete instructor page

* user schema

* indentation

* indentation

* explore publish and taskview sidenav

* explore view links to course/courseId

* taskview sidenav

* taskview

* taskview sidenav changes

* Extend terminal on small width

* .

* .

* Added UserList Component in Courses

* Edit Course

* .

* course dashboard get courseid from url

* delete userlist

* input

* delete userlist

* publish for global admin

* Fixed router params

* Issue 161 fixed: instructor field in courses

* .

* fix nextButton

* .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants