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

Allow event-routing-backends to be used when transforming non-openedx events #431

Merged
merged 9 commits into from
Jun 20, 2024

Commits on Jun 12, 2024

  1. fix: catch ImportErrors

    so that event-routing-backends can be dependency for non-edx-platform
    repos like openedx-completion-aggregator.
    pomegranited committed Jun 12, 2024
    Configuration menu
    Copy the full SHA
    da2052a View commit details
    Browse the repository at this point in the history
  2. feat: adds utils.settings.event_tracking_backends_config()

    * Pulls the EVENT_TRACKING_BACKENDS configuration template into a
      utility method so it can be used by other apps.
    * Pulls allowed xAPI and Caliper events into settings to preserve
      changes
    * Adds a test
    pomegranited committed Jun 12, 2024
    Configuration menu
    Copy the full SHA
    a415ee7 View commit details
    Browse the repository at this point in the history

Commits on Jun 13, 2024

  1. fix: event_tracking_backends_config must use the provided plugin sett…

    …ings
    
    instead of global django.conf.settings
    pomegranited committed Jun 13, 2024
    Configuration menu
    Copy the full SHA
    47c25fb View commit details
    Browse the repository at this point in the history
  2. fix: typos

    in class variable name and in a comment.
    pomegranited committed Jun 13, 2024
    Configuration menu
    Copy the full SHA
    5d42bf7 View commit details
    Browse the repository at this point in the history
  3. refactor: creates fixture test mixins that can be used outside of ERB

    Refactors the transformer test classes to split the test functionality into two:
    
    * TransformersFixturesTestMixin -- for running the event fixture tests
    * TransformersTestMixin -- for running the ERB-specific event tests
    
    The fixtures file path constants have been moved to property methods so
    they can be overrideen when used outside of ERB.
    pomegranited committed Jun 13, 2024
    Configuration menu
    Copy the full SHA
    6e383b9 View commit details
    Browse the repository at this point in the history
  4. fix: catch FileNotFoundError in test mixin

    This exception may occur when using ERB transformer test mixins because
    the fixture test files are not installed with this package.
    pomegranited committed Jun 13, 2024
    Configuration menu
    Copy the full SHA
    f35c58a View commit details
    Browse the repository at this point in the history

Commits on Jun 20, 2024

  1. fix: ensure plugin loading order doesn't matter for event routing set…

    …tings
    
    Initialize EVENT_TRACKING_BACKENDS_ALLOWED_XAPI_EVENTS and
    EVENT_TRACKING_BACKENDS_ALLOWED_CALIPER_EVENTS only if they aren't
    already initialized, and append our events to them.
    
    This allows other plugins to modify these settings too.
    pomegranited committed Jun 20, 2024
    Configuration menu
    Copy the full SHA
    c5a89fa View commit details
    Browse the repository at this point in the history
  2. test: test the "append to events settings" functionality

    and fix coverage
    pomegranited committed Jun 20, 2024
    Configuration menu
    Copy the full SHA
    dc3a0f2 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    7ebd041 View commit details
    Browse the repository at this point in the history