Skip to content

Feature Request: Specify Target LRS for transform_tracking_logs Command #483

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.

Sign up for GitHub

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

Jump to bottom
Closed
AlmogGal1 opened this issue Jan 22, 2025 · 0 comments · Fixed by #485
Closed

Feature Request: Specify Target LRS for transform_tracking_logs Command #483

AlmogGal1 opened this issue Jan 22, 2025 · 0 comments · Fixed by #485
Assignees
@Ali-Salman29
Labels
core contributor PR author is a Core Contributor (who may or may not have write access to this repo).

Comments

@AlmogGal1
Copy link

We are trying to use the transform_tracking_logs command to bulk-load historical data into our edX Aspects environment, with Ralph as the LRS.
We would like the ability to specify a particular LRS to send the data to (e.g., using a flag such as --LRS) so that the data is sent exclusively to Ralph.
Currently, the command does not support this functionality, which means all tracking logs are sent to all configured LRSs. This is a problem for us, as we do not want to spam the existing LRSs with tracking logs spanning several years.
@github-project-automation github-project-automation bot moved this to Needs Triage in Contributions Jan 30, 2025
Ali-Salman29 added a commit to Ali-Salman29/event-routing-backends that referenced this issue Jan 30, 2025
@Ali-Salman29
feat: add --lrs-urls option to transform_tracking_logs command
e3caa9c 
Introduced the --lrs-urls argument to allow specifying multiple LRS route URLs.
This enhancement enables sending tracking logs exclusively to the designated LRSs,
preventing data from being sent to all configured LRSs.

close openedx#483
@Ali-Salman29 Ali-Salman29 mentioned this issue Jan 30, 2025
Merged
8 tasks
Ali-Salman29 added a commit to Ali-Salman29/event-routing-backends that referenced this issue Feb 6, 2025
@Ali-Salman29
feat: Add --lrs-urls option to transform_tracking_logs command
6afb9c3 
- Introduced the --lrs-urls flag to specify target Learning Record Stores (LRS) by their route_url.
- Updated documentation to reflect the new usage of the --lrs-urls option.
- Added tests to ensure correct functionality when using the --lrs-urls flag.

close openedx#483
Ali-Salman29 added a commit to Ali-Salman29/event-routing-backends that referenced this issue Feb 6, 2025
@Ali-Salman29
feat: Add --lrs-urls option to transform_tracking_logs command
279dbbb 
- Introduced the --lrs-urls flag to specify target Learning Record Stores (LRS) by their route_url.
- Updated documentation to reflect the new usage of the --lrs-urls option.
- Added tests to ensure correct functionality when using the --lrs-urls flag.

close openedx#483
@github-project-automation github-project-automation bot moved this from Needs Triage to Done in Contributions Feb 6, 2025
@awais786 awais786 added the core contributor PR author is a Core Contributor (who may or may not have write access to this repo). label Mar 12, 2025
@Ali-Salman29 Ali-Salman29 moved this from Todo to In Progress in Open edX community contributions Mar 13, 2025
@Ali-Salman29 Ali-Salman29 moved this from In Progress to Done in Open edX community contributions Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ali-Salman29 Ali-Salman29

Labels
core contributor PR author is a Core Contributor (who may or may not have write access to this repo).
Projects
Contributions
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add --lrs-urls option to transform_tracking_logs command Ali-Salman29/event-routing-backends
3 participants
@awais786 @Ali-Salman29 @AlmogGal1