-
Notifications
You must be signed in to change notification settings - Fork 1
DLSV2-443 Signposting parameters flow #850
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
DLSV2-443 Signposting parameters flow #850
Conversation
…ning portal and self assessment question
…gation, application title.
…ing session when going back, enabling Skip button, etc
…ng back, reset min and max values to the original question values or the question selected by user, then set min and max result for the parameter in session.
… an assessment question.
…ing signposting Status (essential/recommended) and final Confirmation Summary form, updating or insering changes into parameters table
…SV2-443-SignpostingParametersFlow
…rameter flow. Removed unnecessary code and fixed problem with one "change" link on final summary confirmation form.
…elf-assessment results.
…SV2-443-SignpostingParametersFlow
… when option type is checkboxes and use previous version for slider. Fixed issues with add resource button missing when no resources found. Use competency name instead of description.
… id was expected. Also, after adding a CompetencyLearningResource, redirect to Signposting for the newly created resource.
…ld work anyway as controller method not using this paramter but session previously created in StartSignpostingParametersSession.
…ns. Allow to store parameters with none selected question. Minor changes on user interface.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic. This is working very well. A few minor changes requested in comments.
...121604_AllowNullAssessmentQuestionIdOnTableCompetencyResourceAssessmentQuestionParameters.cs
Show resolved
Hide resolved
DigitalLearningSolutions.Data/Models/Frameworks/SignpostingResourceParameter.cs
Outdated
Show resolved
Hide resolved
DigitalLearningSolutions.Web/Views/Frameworks/Developer/EditCompetencyLearningResources.cshtml
Outdated
Show resolved
Hide resolved
...alLearningSolutions.Web/Views/Frameworks/Developer/SignpostingParametersSetStatusView.cshtml
Show resolved
Hide resolved
DigitalLearningSolutions.Web/Views/Frameworks/Developer/SignpostingSetStatus.cshtml
Outdated
Show resolved
Hide resolved
DigitalLearningSolutions.Web/Views/Frameworks/Developer/SignpostingSetStatus.cshtml
Outdated
Show resolved
Hide resolved
DigitalLearningSolutions.Web/Views/Frameworks/Developer/_SignpostingCard.cshtml
Outdated
Show resolved
Hide resolved
DigitalLearningSolutions.Web/Views/Frameworks/Developer/_SignpostingCard.cshtml
Outdated
Show resolved
Hide resolved
…https://github.com/TechnologyEnhancedLearning/DLSV2 into Develop/Features/DLSV2-443-SignpostingParametersFlow
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. Almost perfect, now. One last scenario that isn't handled quite right... If I have a competency with either an assessment question with related competency assessment question role requirements or two assessment questions and I skip selecting an assessment question in the first step of editing parameters, I am still shown the "Compare result to" step of the flow (although my selection here isn't shown on the summary). This step should be skipped if the user hasn't chosen an asssessment question and they should be taken straight to the set status step.
…cks Next after choosing that question, go to SignpostingSetStatus view.
…https://github.com/TechnologyEnhancedLearning/DLSV2 into Develop/Features/DLSV2-443-SignpostingParametersFlow
… show that text instead of relevance question
…g on role requirements and number of questions
JIRA link
https://hee-dls.atlassian.net/browse/DLSV2-443
Description
Developed multi-step flow for creating or updating learning resource parameters. User choices are stored in session and the application implements user friendly urls throgout the process.
Screenshots
Select a resource and press Manage Parameter or Add
Depending on the chosen option, user is shown sliders or checkboxes in the next stage
Select max and min values with sliders
Select max and min values with checkboxes
Select a compare result question
Parameter is essential or recommended
Final confirm form summary
Flow and views involved
Most relevant tables in database and their relations
Decision tree
Developer checks