Add ancestor information to summary and session log exports

thesujai commented 2 weeks ago

Summary

Adds the ancestor information(ContentNode title) of each content to log exports before writing into csv

References

Fixes #12691

Reviewer guidance

Testing checklist

[x] Contributor has fully tested the PR manually
[ ] If there are any front-end changes, before/after screenshots are included
[ ] Critical user journeys are covered by Gherkin stories
[ ] Critical and brittle code paths are covered by unit tests

PR process

[x] PR has the correct target branch and milestone
[ ] PR has 'needs review' or 'work-in-progress' label
[ ] If PR is ready for review, a reviewer has been added. (Don't use 'Assignees')
[ ] If this is an important user-facing change, PR or related issue has a 'changelog' label
[ ] If this includes an internal dependency change, a link to the diff is provided

Reviewer checklist

PR is fully functional
PR has been tested for accessibility regressions
External dependency files were updated if necessary (yarn and pip)
Documentation is updated
Contributor is in AUTHORS.md

thesujai commented 2 weeks ago

@nucleogenesis @rtibbles A few more things: 1) Should we cache the max_ancestor_depth in get_max_ancestor_depth()? 2) Are unit tests required for this PR?

github-actions[bot] commented 2 weeks ago

Build Artifacts

Asset type	Download link
PEX file	kolibri-0.18.0.dev0_git.20241106170722.pex
Windows Installer (EXE)	kolibri-0.18.0.dev0+git.20241106170722-windows-setup-unsigned.exe
Debian Package	kolibri_0.18.0.dev0+git.20241106170722-0ubuntu1_all.deb
Mac Installer (DMG)	kolibri-0.18.0.dev0+git.20241106170722.dmg
Android Package (APK)	kolibri-0.18.0.dev0+git.20241106170722-0.1.4-debug.apk
TAR file	kolibri-0.18.0.dev0+git.20241106170722.tar.gz
WHL file	kolibri-0.18.0.dev0+git.20241106170722-py2.py3-none-any.whl

thesujai commented 2 weeks ago

It is super optimized now(maybe)

thesujai commented 2 weeks ago

@rtibbles Updated! if it can be optimized more let me know

learningequality / kolibri