Skip to content

chore: add logging for sending course completion xAPI statements#92

Open
msaha-sonata wants to merge 6 commits intoopenedx:mainfrom
msaha-sonata:maha/ENT-11229
Open

chore: add logging for sending course completion xAPI statements#92
msaha-sonata wants to merge 6 commits intoopenedx:mainfrom
msaha-sonata:maha/ENT-11229

Conversation

@msaha-sonata
Copy link
Copy Markdown
Contributor

Description:
Logging added for capture sending course completion xAPI statements

Jira:
ENT-11229

Copy link
Copy Markdown

@sjasti-sonata-svg sjasti-sonata-svg left a comment

Choose a reason for hiding this comment

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

This branch is currently 4 commits behind main.
Please rebase (or merge) main so the PR is up to date before requesting external review.
Please add Jira ticket number to the PR description.

Comment thread CHANGELOG.rst Outdated
Comment on lines +41 to +43

=======

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please remove

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

@msaha-sonata
Copy link
Copy Markdown
Contributor Author

@sjasti-sonata-svg
Branch is up to date. Jira ticket number is already in the PR description.
Can you please check and approve. Thanks.

Copy link
Copy Markdown

@sjasti-sonata-svg sjasti-sonata-svg left a comment

Choose a reason for hiding this comment

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

As per required changes done by @msaha-sonata it is Approved.

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.

4 participants