Skip to content

[SE-3248] Revert progress_video event changes - #341

Merged
toxinu merged 1 commit into
opencraft-release/koa.3from
toxinu/revert-se-3248
Apr 30, 2021
Merged

[SE-3248] Revert progress_video event changes#341
toxinu merged 1 commit into
opencraft-release/koa.3from
toxinu/revert-se-3248

Conversation

@toxinu

@toxinu toxinu commented Apr 19, 2021

Copy link
Copy Markdown

This pull request is about reverting #256 and #266 since we won't merge those into upstream (https://github.com/edx/edx-platform/pull/25038).

Jira ticket: SE-3248

Testing instructions: Manually check that the changes have been reverted by doing a diff.

Reviewers: @nizarmah

@toxinu toxinu self-assigned this Apr 19, 2021
@toxinu toxinu changed the title [SE-3248] Revert changes [SE-3248] Revert progress_video event changes Apr 20, 2021
@toxinu
toxinu requested a review from nizarmah April 20, 2021 00:36

@nizarmah nizarmah left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@toxinu

👍

  • I tested this by diffing the changes against the pull requests mentioned in the commit
  • I read through the code
  • I checked for accessibility issues
  • Includes documentation
  • I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository.

@toxinu
toxinu changed the base branch from opencraft-release/koa.2a to opencraft-release/koa.3 April 23, 2021 05:50
@toxinu
toxinu changed the base branch from opencraft-release/koa.3 to opencraft-release/koa.2a April 23, 2021 05:51
@toxinu
toxinu changed the base branch from opencraft-release/koa.2a to autodesk-release/koa.3 April 28, 2021 01:53
@toxinu
toxinu force-pushed the toxinu/revert-se-3248 branch from c0a8488 to b81a790 Compare April 28, 2021 01:55
@toxinu
toxinu changed the base branch from autodesk-release/koa.3 to opencraft-release/koa.3 April 28, 2021 01:56
@toxinu
toxinu force-pushed the toxinu/revert-se-3248 branch from b81a790 to 95cb83b Compare April 28, 2021 01:56
@toxinu
toxinu force-pushed the toxinu/revert-se-3248 branch from 95cb83b to b0054b9 Compare April 30, 2021 02:51
@toxinu
toxinu merged commit ca4dd72 into opencraft-release/koa.3 Apr 30, 2021
@toxinu
toxinu deleted the toxinu/revert-se-3248 branch April 30, 2021 02:51
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.

2 participants