Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MVJ-470 Avoid PicklingError in email reports #753

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Commits on Oct 22, 2024

  1. Configuration menu
    Copy the full SHA
    5232752 View commit details
    Browse the repository at this point in the history
  2. Avoid PicklingError

    Passing complex objects to django-q async_task results in a
    PicklingError during serialization which prevents report creation.
    This change avoids the problem by only passing module-level functions
    and regular Python data structures to async_task and between the
    asynchronous function and the hook.
    juho-kettunen-nc committed Oct 22, 2024
    Configuration menu
    Copy the full SHA
    82e04ea View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    aa3acb4 View commit details
    Browse the repository at this point in the history

Commits on Oct 23, 2024

  1. Increase timeout on rent forecast report

    Without this property, the async task would use the default timeout, which is currently too short at 90 seconds.
    juho-kettunen-nc committed Oct 23, 2024
    Configuration menu
    Copy the full SHA
    189efe1 View commit details
    Browse the repository at this point in the history
  2. Revert "Use full import paht in async_task call"

    This reverts commit aa3acb4.
    
    Trying out whether the change was at all needed
    juho-kettunen-nc committed Oct 23, 2024
    Configuration menu
    Copy the full SHA
    7540a55 View commit details
    Browse the repository at this point in the history
  3. Add custom timeout to lease statistic report

    This is just in case, because async email reports appear to sometimes
    get eternally stuck in the queues if the worker is killed by a timeout.
    juho-kettunen-nc committed Oct 23, 2024
    Configuration menu
    Copy the full SHA
    22d1684 View commit details
    Browse the repository at this point in the history