Skip to content

Add information about RST keywords for course settings#74

Open
kinnala wants to merge 1 commit intoapluslms:masterfrom
kinnala:master
Open

Add information about RST keywords for course settings#74
kinnala wants to merge 1 commit intoapluslms:masterfrom
kinnala:master

Conversation

@kinnala
Copy link

@kinnala kinnala commented Apr 24, 2025

Description

Adding a note about changing course settings via RST commands.

Why?

The current manual reads like one should use Edit course user interface to set deadlines. However, those options might get overridden by recompilation.

How?

Added a note in the beginning of the page "Course settings"

There is no issue representing this since I believe it is a small change.

Testing

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

What type of test did you run?

  • Accessibility test using the WAVE extension.
  • Django unit tests.
  • Selenium tests.
  • Other test. (Add a description below)
  • Manual testing.

[ADD A DESCRIPTION ABOUT WHAT YOU TESTED MANUALLY]

Did you test the changes in

  • Chrome
  • Firefox
  • This pull request cannot be tested in the browser.

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

Translation

Programming style

  • Did you follow our style guides?
  • Did you use Python type hinting in all functions that you added or edited? (type hints for function parameters and return values)

Have you updated the README or other relevant documentation?

  • documents inside the doc directory.
  • README.md.
  • Aplus Manual.
  • Other documentation (mention below which documentation).

Is it Done?

  • Reviewer has finished the code review
  • After the review, the developer has made changes accordingly
  • Customer/Teacher has accepted the implementation of the feature

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant