slatefoundation/slate-cbl

slate-cbl's Issues

JS error when opening demonstration window for second time with a competency pre-selected

implement growth-based promotion

Remove refresh button on student dashboard & automatically refresh the window when a new content area is selected

Mock up Task Tree component for student task view

Theme standard complete checkboxes for student dashboard

Checkbox indicators have been enabled next to each student standard to indicate where all required demonstrations have bene completed:

Positioning needs to be fixed, and the is-checked class needs to be honored

Comment box editing error

From Sydney: "When I click on the Comment box in attempt to revise an existing comment, a grey space takes over most of the text field."

Video:
https://www.dropbox.com/s/v2q8f0366ytchci/SLATE%20-%20Comment%20Box%20UI%20Issue.mov?dl=0

Mock up clear item functionality for TodoList

Mock up Filter menu for TaskTree

The standard overview window doesn't update automatically after deletion

Steps to reproduce:

1. Open teacher dashboard
2. Click a cell to open a standard overview window
3. Delete a demonstration

Actual Results: The teacher dashboard UI refreshes and removes those filled in cells, but it remains in the list. If you click delete again, you get a server error.

Expected Result: Demonstration should be removed from the list in the standard overview window

merge ryon's style tweaks in from rc.v1.slate.is

upgrade to ext 5.1 or 6.0

Logged demonstrations not showing

In some cases, multiple logs are not being displayed and the cell is left blank. When a new log is made, that log shows up but with no background color. The open window in the screen shot is for the third standard in the open competency, which is blank.

Support collapsing of SimplePanel

refactor student dashboard sass

The SASS for the student dashboard is in site-root/sass/student-dashboard.scss, making it part of the site's compass build system rather than the CBL tool's and preventing it from using the same variables as the teacher dashboard

Since the student dashboard has been converted to components in the slate-cbl package (Slate.cbl.view.student.Dashboard, Slate.cbl.view.student.CompetencyCard, and Slate.cbl.view.student.RecentProgress), the sass should be refactored into the slate-cbl/sass directory, separated by subcomponent

• Delete site-root/sass/student-dashboard.scss
• Revert 4933fc5

Course section selector

Drop down menu lists all sections including previous terms. It should only show sections for the current term.

Scaffold frontend view and data models/stores

Start by replicating the Slate.cbl.view.teacher.StudentsProgressGrid

At some point the existing competency dashboard views will need their namespaces/names shuffled around to coexist in the same package with these new task dashboard views. Sharing some code and templates might be possible too, but let's start with copy+paste and taking full liberty to change things as needed, and only look back at the end to consider what similar things should be merged.

The expandy arrows on the left side, narrower columns, and rotated names should all be accomplishable by @ryon without major tweaks to the markup structure

Start out by mockup up model(s) and store(s) needed to drive the tasks dashboard, and populate the tasks store(s) with mockup inline data. Construct mockup data that will exercise as many cases as you can think of so we can see every state shown in the graphical mockups.

Update the rendering pipeline to write relevant state and data into the the grid in place of the competency progress contents. Apply the following CSS classes to each sell according to it's model fields from the store:

• one of:
  • task-due-upcoming
  • task-due-revision
  • task-due-past
• one of:
  • task-submitted-no
  • task-submitted-yes
• one of:
  • task-rated-no
  • task-rated-yes

Write the due (or revision if set) date into a <span class="task-date"> within the cell, and let the CSS handle hiding it if submitted.

Mockup stores for AggregridExample

Define 3 standalone store classes in the AggregridExample app, instantiate them through the stores config for the Application, and pass them into the aggregrid via their global storemgr id

Create a store for columns called StudentsStore, a store for rows called TimePeriodsStore, and a store for data called AbsencesStore

Fill each store with a healthy amount of made up records, make sure columns and rows render

"…" visible in competency search placeholder after blur

Perhaps the unicode character should be used instead of the HTML entity?

Mock up clear/hide buttons for TodoList

Graduate script runs out of memory

When running /cbl/graduate the following error is produced "Fatal error: Allowed memory size of 209715200 bytes exhausted (tried to allocate 81 bytes) in /usr/local/lib/node_modules/emergence/php-bootstrap/lib/Debug.class.php on line 64"

Override to Promotion - rendering issue

When Overriding a number of cells to force a promotion, the display retains the overrides that were logged at the previous level. In the screen shot below, all five of the Overrides for Brittany Clay were logged at level 9 but remained when it rolled over.

After refreshing, all are dropped except the final Override that was logged (in this case Cite Evidence).

The log is counting as a level 10 override in the Standard Overview window.

student dashboard doesn't load

create issues for remaining phase 2 backend

Cell colors not rendering

Default colors not rendering consistently. For some students the colors appear but not for others. Example attached.

Evidence requirement boxes become one big box after logging a demonstration in that standard

Steps to reproduce:

1. Navigate to the teacher dashboard and filter to a group and content area.
2. Click an empty evidence requirement box/cell
3. Click log a demonstration
4. Add another competency to the demonstration & rate at least one standard in each.
5. Uncheck load next student

Actual Result:
While the main competency % complete and progress bar updates, the evidence requirements box/cell becomes one long one. (Only happens for the initial standard selected for log a demo)

Expected Result:
Evidence requirement cells should retain their dividers and the first box should fill with the info from the recently logged demo.

implement standard overrides

upgrade cbl-dev to slate-v2

# of demonstrations doesn't agree with Competency Overview window

When viewing details of a skill already completed at level 10, only three demonstrations show, which is clearly incorrect.

In this example, the first two demonstrations should have been logged on level 9 and then only the override at level 10.

In this example, two logs show in the Competency Overview, but one shows at level 10, which should mean that 3 have been logged.

merge testing suite with ryon's work

average-too-low logic uses static 8.5 minimum average

Mock up To-Do List component for student task view

audit codebase

• Search for TODO comments
• Strip frontend console.log calls
• Find and remove deprecated API endpoints
• Audit controller permissions
• Eliminate old PHP array syntax
• Eliminate leading commas in PHP and JS
• Inspect dev-cbl for unmerged upstream overrides
• Sweep trailing commas and whitespace

Mock up Simple Panel component for Slate

Mock up "new task" input for TodoList

create migration script to graduate already-ready students to level 10

move completion evaluation code in DemonstrationSkill/afterRecordSave handler to a Competency member function that the migration script can use too.

Maybe instead of a migration script it is a simple tool any admin can use to:

• Scan for and list students eligible for advancements they haven't gotten
• Scan for and list students with advancements they no longer qualify for
• Auto-advance students with missing advancements
• In the future: Un-advance students with missing qualifications

Incorrect calculations

Percentage complete is incorrect for a few reasons

1. M's are being counted as completions
   ex. Engaging in collaboratie discussions 42% for both when some are M's and some aren't
2. Fully complete competencies are being counted as less than 100%
   ex. Writing Narratives texts 88% vs. 100% when both are full

More examples here

/cbl/graduate doesn't check for account level

Should require staff access or above but does not currently.

missed demonstrations are getting level colors

They should be grayed out and not show their target level's color

Wrong (from dev-cbl):

Right (from cbl.v1):

Server error

When loading a Standard Overview, a 403 server error results. The window opens, but the server error window also appears but does not show the details.

demonstration delete button does nothing

rename skills to standards system-wide

The term "skill" and "standard" are currently used interchangeably throughout the system, making things a minefield for future developers. Skill was the original term but has been deprecated in favor of Standard.

PHP, JS, and UI code must be refactored and database migrations written and tested.

improve handling of dashboards when students not enrolled

The teacher and student dashboards are unpleasant when you try to view students+competencies that have no enrollments

student dashboard needs level switching buttons

• Add temporary button elements
• Wire buttons
• Get buttons styled

export scripts need to be setup at site-root/cbl/exports/

Test for compatibility with levels and make minimal changes where needed

Scaffold backend data model for tasks

Create backend ActiveRecord and RecordsRequestHandler implementations under the Slate\CBL\Tasks\ namespace:

• \Slate\CBL\Tasks\Task
  • extends \VersionedRecord
  • model after \Slate\CBL\Demonstrations\Demonstration
• \Slate\CBL\Tasks\ExperienceTask
  • extends Task
  • model after \Slate\CBL\Demonstrations\ExperienceDemonstration
• \Slate\CBL\Tasks\TasksRequestHandler
  • extends \RecordsRequestHandler
  • model after \Slate\CBL\Demonstrations\DemonstrationsRequestHandler

Task fields

Title

• type: string

Handle

• type: string
• unique: true

ParentTaskID

• type: integer
• default: null

DueDate

• type: timestamp
• default: null

ExpirationDate

• type: timestamp
• default: null

Instructions

• type: clob
• default: null

Shared

• type: enum('course', 'school', 'public')
• default: null

ExperienceTask fields

ExperienceType

• type: string

UI Mockups

Create a Task

Edit a Task

setup siesta testing suite

• setup basic siesta sanity test to verify framework is loaded
• create series of tests that check that rowsStore and columnsStore return correct counts and the update event has been fired exactly once with every combination of instantiation, store configuration, and rendering happeing in different orders and groupings

Merge New Reports upstream

Demonstrations and demonstrations-legacy reports have been updated to include demonstration IDs. demonstrations-legacy has also had teacher name added for purposes of queries that B21 wants to make about task completion percentages by standard and by teacher.

These changes have also been made on Allentown, but should be merged upstream and pushed back down to all CBL instances.

Demonstrations-legacy code: http://staging.fr.b21.slatepowered.net/develop#/_parent/site-root/cbl/exports/demonstrations-legacy.php

Demonstrations code: http://staging.fr.b21.slatepowered.net/develop#/site-root/cbl/exports/demonstrations.php

Completion Percentages not Registering

Logged demonstrations appear in the grid but do not count towards competency completion. Color rendering also is an issue here, which may be related.

Transition brings some scores forward into the next level

Recreate by completing all evidence requirements in a competency at low levels so that the level does not advance. Then, log new demonstrations at a high level to increase the average score. For example, log all two 8s for each standard in a competency and then log a third demonstration with every standard as a 12.

Result: Some of the demonstrations carry over to the next level.

0% completion renders blank

No background in a cell when there has not been a log. The background should still be the level color.