Repository navigation

feat: add endpoint to delete or archive a CompetencyCriterion (#674) - #879

Open

ufedaseyeuconsultant wants to merge 24 commits into

openedx:mainfrom

ufedaseyeuconsultant:ufedaseyeu/674-delete-competency-criterion

Open

feat: add endpoint to delete or archive a CompetencyCriterion (#674)#879
ufedaseyeuconsultant wants to merge 24 commits into
openedx:mainfrom
ufedaseyeuconsultant:ufedaseyeu/674-delete-competency-criterion

Commits on Nov 2, 2026
Commits on Nov 4, 2026
- fix: serialize criterion writers on the competency's Tag row (#674)
  
  ufedaseyeuconsultant
  and
  claude
  committed
  0a717a2
- test: cover criterion delete races over two MySQL connections (#674)
  
  ufedaseyeuconsultant
  and
  claude
  committed
  8d66a52
- fix: record the acting user on deletion history rows (#674)
  
  ufedaseyeuconsultant
  and
  claude
  committed
  a44fb5b
- refactor: drop the cascade's event parameter, fix lock comments (#674)
  
  ufedaseyeuconsultant
  and
  claude
  committed
  d0a8f77
- test: assert real behavior instead of stubbed tagging reads (#674)
  
  ufedaseyeuconsultant
  and
  claude
  committed
  cfa1710
- docs: record #674's deviations in ADRs 0002, 0003 and 0004 (#674)
  
  ufedaseyeuconsultant
  and
  claude
  committed
  5c767e9
- docs: tighten the new lock docstrings and comments (#674)
  
  ufedaseyeuconsultant
  and
  claude
  committed
  255f891
- fix: lock in create call, map deleted tag in group view
  
  ufedaseyeuconsultant
  and
  claude
  committed
  c620f1d
- fix: check permission before locking in group update
  
  ufedaseyeuconsultant
  and
  claude
  committed
  3e9a232
- refactor: set history users through one helper
  
  ufedaseyeuconsultant
  and
  claude
  committed
  be29b8f
- docs: tighten docstrings, comments and ADR wording for criterion delete
  
  ufedaseyeuconsultant
  and
  claude
  committed
  2dedf27
- test: pin lock points and tighten assertions for criterion removal
  
  ufedaseyeuconsultant
  and
  claude
  committed
  3bb7e6d

Navigation Menu

Repository navigation

feat: add endpoint to delete or archive a CompetencyCriterion (#674) - #879

feat: add endpoint to delete or archive a CompetencyCriterion (#674)#879
ufedaseyeuconsultant wants to merge 24 commits into
openedx:mainfrom
ufedaseyeuconsultant:ufedaseyeu/674-delete-competency-criterion

Commits on Nov 2, 2026

feat: add endpoint to delete or archive a CompetencyCriterion (#674)

fix: decide hard-delete or archive inside the transaction (#674)

fix: archive a refused delete only when the criterion has a status (#674)

refactor: check permission inside delete_or_archive_competency_criterion (#674)

fix: lock tag, ancestors, and criterion instead of retrying (#674)

fix: write only the archived flag, record the user, stop at archived levels (#674)

fix: untag only the object tag the tagging API reports (#674)

docs: correct the delete view and cascade docstrings and tidy the tests (#674)

fix: map validation errors in the delete view and read the lineage under the tag lock (#674)

fix: remove only the orphaned object tag when deleting a criterion (#674)

fix: lock only the tag when deleting a criterion (#674)

fix: delete an emptied archived group on a criterion delete (#674)

Commits on Nov 4, 2026

fix: serialize criterion writers on the competency's Tag row (#674)

test: cover criterion delete races over two MySQL connections (#674)

fix: record the acting user on deletion history rows (#674)

refactor: drop the cascade's event parameter, fix lock comments (#674)

test: assert real behavior instead of stubbed tagging reads (#674)

docs: record #674's deviations in ADRs 0002, 0003 and 0004 (#674)

docs: tighten the new lock docstrings and comments (#674)

fix: lock in create call, map deleted tag in group view

fix: check permission before locking in group update

refactor: set history users through one helper

docs: tighten docstrings, comments and ADR wording for criterion delete

test: pin lock points and tighten assertions for criterion removal