inducer / inducer/relate

Optional pages should be allowed to have a `value`

Open
#935 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
435
Forks
124
Avg merge
5h 40m
Merged PRs (30d)
14

Description

Currently, they can't:

https://github.com/inducer/relate/blob/e21c5ac9923e086de2703ce41e02e7e900f6140a/course/page/base.py#L829-L834=

There's an outright contradiction here, since the upload question spews a warning complaining about lack of `value`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in course/page/base.py at the linked lines 829-834 and inspect how optional pages handle value. Compare that behavior with the upload question warning mentioned in the issue; done means optional pages can have a value without the contradictory lack-of-value warning.

Written by the indexing model from the issue text.

Assessment

Tech stack
django, python
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.