96f419d653
Audit of the admin Configuration section (FAQ Manager, Notification Rules,
Approval Config) surfaced three contract mismatches and one missing schema.
FAQ
* Backend `encoach.faq.category.audience` / `encoach.faq.item.audience`
rejected the `both` and `entity` values emitted by the FAQ Manager
UI. Widened the Selection so the UI's full vocabulary round-trips.
* FAQ item `video_url` was already accepted by the UI but silently
dropped by the controller; now persisted via a new `video_url`
field on `encoach.faq.item` and serialized on list/get responses.
Notification Rules
* Added `days_before`, `frequency`, `channel`, `entity_id` to
`encoach.notification.rule`. These are the exact fields the admin
form collects; prior to this change they were serialized into the
JSON body, ignored by the controller, and omitted from responses,
leaving the Active switch permanently off and the table columns
blank.
* Controller now translates `active` <-> `is_active` both ways so the
new frontend contract and legacy callers coexist. Missing required
fields return HTTP 400 instead of 500.
Approval Workflows
* The controller had been hand-rolling raw SQL against three tables
(`encoach_approval_workflow`, `_stage`, `_request`) that no Odoo
model declared, so the tables never existed. List() was guarded and
returned empty; create() would 500 with "relation does not exist".
* Introduced real ORM models in `encoach_exam_template/models/approval.py`
plus access rights, which auto-provision the tables on -u.
* Rewrote the controller to use ORM, added PATCH, and emitted both
`items` and `results` in the list envelope so the frontend's
PaginatedResponse reader and legacy callers both work. Step
payloads now carry `max_days`, `auto_escalate`, and
`notification_email` end to end.
* Frontend `approvalsService` and `ApprovalWorkflowConfig` updated to
send the full stage shape + `allow_bypass`, tolerate both envelope
keys, and validate at least one approver before submit.
Schema delta applied via `./run.sh -u encoach_exam_template,encoach_lms_api`.
Verified with new `test_config_flows.py`: 24/24 passing. Regression runs
on `test_support_flows.py` (29/29) and `test_training_flows.py` (26/26)
remain green.
Made-with: Cursor
2.2 KiB
2.2 KiB
| 1 | id | name | model_id:id | group_id:id | perm_read | perm_write | perm_create | perm_unlink |
|---|---|---|---|---|---|---|---|---|
| 2 | access_encoach_exam_template_user | encoach.exam.template.user | model_encoach_exam_template | base.group_user | 1 | 1 | 1 | 1 |
| 3 | access_encoach_passage_user | encoach.passage.user | model_encoach_passage | base.group_user | 1 | 1 | 1 | 1 |
| 4 | access_encoach_audio_file_user | encoach.audio.file.user | model_encoach_audio_file | base.group_user | 1 | 1 | 1 | 1 |
| 5 | access_encoach_question_user | encoach.question.user | model_encoach_question | base.group_user | 1 | 1 | 1 | 1 |
| 6 | access_encoach_writing_prompt_user | encoach.writing.prompt.user | model_encoach_writing_prompt | base.group_user | 1 | 1 | 1 | 1 |
| 7 | access_encoach_speaking_card_user | encoach.speaking.card.user | model_encoach_speaking_card | base.group_user | 1 | 1 | 1 | 1 |
| 8 | access_encoach_rubric_user | encoach.rubric.user | model_encoach_rubric | base.group_user | 1 | 1 | 1 | 1 |
| 9 | access_encoach_rubric_group_user | encoach.rubric.group.user | model_encoach_rubric_group | base.group_user | 1 | 1 | 1 | 1 |
| 10 | access_encoach_exam_custom_user | encoach.exam.custom.user | model_encoach_exam_custom | base.group_user | 1 | 1 | 1 | 1 |
| 11 | access_encoach_exam_custom_section_user | encoach.exam.custom.section.user | model_encoach_exam_custom_section | base.group_user | 1 | 1 | 1 | 1 |
| 12 | access_encoach_exam_assignment_user | encoach.exam.assignment.user | model_encoach_exam_assignment | base.group_user | 1 | 1 | 1 | 1 |
| 13 | access_encoach_exam_schedule_user | encoach.exam.schedule.user | model_encoach_exam_schedule | base.group_user | 1 | 1 | 1 | 1 |
| 14 | access_encoach_exam_structure_user | encoach.exam.structure.user | model_encoach_exam_structure | base.group_user | 1 | 1 | 1 | 1 |
| 15 | access_encoach_student_attempt_user | encoach.student.attempt.user | model_encoach_student_attempt | base.group_user | 1 | 1 | 1 | 1 |
| 16 | access_encoach_student_answer_user | encoach.student.answer.user | model_encoach_student_answer | base.group_user | 1 | 1 | 1 | 1 |
| 17 | access_encoach_student_score_user | encoach.student.score.user | model_encoach_student_score | base.group_user | 1 | 1 | 1 | 1 |
| 18 | access_encoach_approval_workflow_user | encoach.approval.workflow.user | model_encoach_approval_workflow | base.group_user | 1 | 1 | 1 | 1 |
| 19 | access_encoach_approval_stage_user | encoach.approval.stage.user | model_encoach_approval_stage | base.group_user | 1 | 1 | 1 | 1 |
| 20 | access_encoach_approval_request_user | encoach.approval.request.user | model_encoach_approval_request | base.group_user | 1 | 1 | 1 | 1 |