Skip to content
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

1318 - Add Learning Objectives section in sidebar #1319

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

legendword
Copy link
Contributor

Changes

  • Add node.learningObjectives property that stores the learning objectives the author set.
  • Show the Learning Objectives information under the node description in the sidebar.
  • Fix bug for the "show title in page" checkbox in the node modal that incorrectly displays as unchecked when the option is by default checked.

Screenshot

  • To be added.

Issue Linkage

Closes #1318

PR Dependency

Depends on: N/A

Automated Testing

  • To be added.

@cypress
Copy link

cypress bot commented May 5, 2023

1 flaky tests on run #4010 ↗︎

0 99 0 0 Flakiness 1

Details:

Merge branch 'master' into 1318-learning-objectives
Project: tapestry-wp Commit: 1553438cb6
Status: Passed Duration: 06:01 💡
Started: May 16, 2023 10:40 PM Ended: May 16, 2023 10:46 PM
Flakiness  cypress/integration/node-authoring.spec.js • 1 flaky test

View Output Video

Test Artifacts
Node Authoring > should not be able to delete a non-leaf node Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@legendword legendword added the almost done This PR needs just under an hour of work to be done label May 8, 2023
@legendword legendword added deferred We have decided not to work on this until a later date (taking too long, etc.) and removed almost done This PR needs just under an hour of work to be done labels May 16, 2023
@legendword
Copy link
Contributor Author

Waiting for discussion on creating effective Learning Objectives and how to integrate Learning Objectives and Learning Outcomes with the current tapestry plugin. Until then, this pull request is on hold.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deferred We have decided not to work on this until a later date (taking too long, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Learning Objectives section in sidebar
1 participant