Skip to content

fix: implement __repr__ method in order to avoid maximum recursion er…#67

Merged
johanseto merged 1 commit intoopen-release/teak.nelpfrom
jlc/teak-mig/Futurex-1257
Oct 11, 2025
Merged

fix: implement __repr__ method in order to avoid maximum recursion er…#67
johanseto merged 1 commit intoopen-release/teak.nelpfrom
jlc/teak-mig/Futurex-1257

Conversation

@johanseto
Copy link
Collaborator

@johanseto johanseto commented Oct 1, 2025

Description

  • fix: implement repr method in order to avoid maximum recursion error

  • chore: apply pr suggestions

(cherry picked from commit 9e26088)

  1. Get into the lms docker container docker exec -it tutor_dev-lms-1 bash
  2. Open a shell ./manage.py lms shell
  3. Run the following code, ensure that you have at least one record
from lms.djangoapps.courseware.models import StudentModuleHistory

StudentModuleHistory.objects.all()[:10]
2025-10-01_15-19

Jira story: https://edunext.atlassian.net/browse/FUTUREX-1257

…ror (#56)

* fix: implement __repr__ method in order to avoid maximum recursion error

* chore: apply pr suggestions

(cherry picked from commit 9e26088)
@johanseto johanseto requested a review from andrey-canon October 1, 2025 20:28
Copy link
Collaborator

@andrey-canon andrey-canon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working as expected

Image

@johanseto johanseto merged commit 8d0c0b4 into open-release/teak.nelp Oct 11, 2025
47 checks passed
@johanseto johanseto temporarily deployed to open-release/teak.nelp October 11, 2025 00:59 — with GitHub Actions Inactive
@johanseto johanseto temporarily deployed to open-release/teak.nelp October 11, 2025 00:59 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants