apluslms / a-plus

A+ frontend portal - A+ LMS documentation:
https://apluslms.github.io/
Other
65 stars 72 forks source link

Make navigation links visible in LTI tool #1290

Closed PasiSa closed 10 months ago

PasiSa commented 10 months ago

Description

What?

This builds on pull request #1275 , and is made against the 1.20 branch. In other words, #1275 should be checked first.

I ended up doing a dedicated _siblings.html template for LTI tool, because the logic is slightly different than in the original one (this supports also the case where LTI is limited to a single module, in which case it will not be possible to navigate to Course content TOC or other modules.)

Why?

[ANSWER HERE]

How?

[ANSWER HERE]

Fixes #

Testing

Remember to add or update unit tests for new features and changes.

What type of test did you run?

[ADD A DESCRIPTION ABOUT WHAT YOU TESTED MANUALLY]

Did you test the changes in

Think of what is affected by these changes and could become broken

Translation

Programming style

Have you updated the README or other relevant documentation?

Is it Done?

Clean up your git commit history before submitting the pull request!