Skip to content

feat: include username in event data#280

Open
BryanttV wants to merge 3 commits intoopen-craft:masterfrom
eduNEXT:bav/add-username-in-event-data
Open

feat: include username in event data#280
BryanttV wants to merge 3 commits intoopen-craft:masterfrom
eduNEXT:bav/add-username-in-event-data

Conversation

@BryanttV
Copy link

Description

This PR adds the username field to the data of the events.

Motivation

Currently, the events emitted by openedx-completion-aggregator only include the user_id, which creates a significant limitation for integration scenarios with external services.

According to the Open edX Events documentation regarding integration with external systems, TRACKING_EVENT_EMITTED events are sent through the event bus to be consumed by third-party services. The issue is that with only the user_id available in the event, external consumers face an incomplete workflow.

This change follows the existing pattern used in Open edX, where events include the username:

Testing Instructions

Using Tutor:

  1. Install the plugin using this branch.
  2. Configure the Event Bus (Redis or Kafka).
  3. Check the emitted event that includes the username field.

Merge Checklist

  • All reviewers approved
  • CI build is green
  • Version bumped
  • Changelog record added
  • Documentation updated (not only docstrings)
  • Commits are squashed
  • PR author is listed in AUTHORS

@BryanttV BryanttV force-pushed the bav/add-username-in-event-data branch from c0d2d16 to 6c91602 Compare January 28, 2026 01:49
@BryanttV BryanttV force-pushed the bav/add-username-in-event-data branch from 6c91602 to 5d89c4d Compare January 28, 2026 15:00
@BryanttV BryanttV marked this pull request as ready for review January 28, 2026 15:00
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.

1 participant