Fix nested field updates and metadata label updates by rtibbles · Pull Request #3377 · learningequality/studio

rtibbles · 2022-04-23T16:02:27Z

Summary

Previous attempts to make dot path updates work as intended did not properly carry out deletes when None was passed
The use of dot paths in the materialized paths in nested metadata labels caused incorrect processing of nested metadata labels
Fixes and adds tests for both of these.

rtibbles added 2 commits April 23, 2022 08:59

Use recursion to properly handle nested extra_fields updates.
c4fc172

Add a dot path exception for metadata label fields.
e6daf76

rtibbles added the TODO: needs review label Apr 23, 2022

rtibbles requested review from bjester and sairina April 23, 2022 16:02

rtibbles added this to the 2022 Q1-Q2 Phase 1: tasks and backlog milestone Apr 23, 2022

bjester approved these changes Apr 25, 2022
View reviewed changes

rtibbles merged commit 0b01b4f into learningequality:unstableApr 25, 2022

rtibbles deleted the metadata_labels_no_depth branch April 25, 2022 15:20

This was referenced Sep 15, 2022
Hotfixes release for development ending 2022-09-16 #3644
Merged
Release v2022.10.03 #3658
Merged