Programs: added platform support package and configuration model. #10350
Conversation
jimabramson
force-pushed
the
jsa/ecom-2657
branch
3 times, most recently
from
085bfbd
to
05bc0e0
Compare
|
@cpennington The new configuration model should be used instead of django settings for Programs API settings, and |
jimabramson
force-pushed
the
jsa/ecom-2657
branch
from
05bc0e0
to
1bb2928
Compare
|
jenkins run python |
| Indicate whether LMS dashboard functionality related to Programs should | ||
| be enabled or not. | ||
| """ | ||
| return self.enabled and self.enable_student_dashboard |
There was a problem hiding this comment.
The way I understand config models, enabled is a sort of master on/off switch. The dashboard-specific toggle is meant to be a kill switch specifically for the learner dashboard. My thinking is that when we roll out Studio integration for example, we may benefit from having individual kill switches for each integration point.
|
👍 |
|
@feanil please review - as discussed in person, this is the working plan for configuring programs IDA URLs in lms/cms, as opposed to using ansible/django settings. |
|
👍 |
| from openedx.core.djangoapps.programs.models import ProgramsApiConfig | ||
|
|
||
|
|
||
| class ProgramsApiConfigAdmin(ConfigurationModelAdmin): # pylint: disable=missing-docstring |
There was a problem hiding this comment.
can we add search to this if we need to find people it will make our lives easier.
There was a problem hiding this comment.
discussed offline - this refers to a separate (future ticket) and n/a here
|
👍 |
Programs: added platform support package and configuration model.
ECOM-2657