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

Fix few frontend bugs slipped in after code restructuring #184

Merged
merged 4 commits into from
Mar 30, 2017

Conversation

z4y4ts
Copy link

@z4y4ts z4y4ts commented Mar 30, 2017

  • Fix Editor tabs switch by passing event to toggleEditorTab() in studio-edit.js.
  • Fix playback state save by correctly reading prev state from localStorage.
  • Add token field to UI for Brightcove and Wistia.

@z4y4ts z4y4ts self-assigned this Mar 30, 2017
@codecov
Copy link

codecov bot commented Mar 30, 2017

Codecov Report

Merging #184 into dev will increase coverage by 0.09%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #184      +/-   ##
==========================================
+ Coverage   71.21%   71.31%   +0.09%     
==========================================
  Files          26       26              
  Lines        1494     1499       +5     
  Branches        2        2              
==========================================
+ Hits         1064     1069       +5     
  Misses        429      429              
  Partials        1        1
Impacted Files Coverage Δ
video_xblock/backends/html5.py 100% <100%> (ø) ⬆️
video_xblock/backends/brightcove.py 54.09% <100%> (+0.42%) ⬆️
video_xblock/backends/wistia.py 95.87% <100%> (+0.13%) ⬆️
video_xblock/backends/base.py 80% <100%> (ø) ⬆️
video_xblock/tests/test_backends.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 42cabe0...cdd6e19. Read the comment docs.

@z4y4ts z4y4ts changed the title Pass event to toggleEditorTab() in studio-edit.js Fix set for Frontend bugs Mar 30, 2017
@z4y4ts z4y4ts changed the title Fix set for Frontend bugs Fix few frontend bugs slipped in after code restructuring Mar 30, 2017
@z4y4ts z4y4ts requested a review from xahgmah March 30, 2017 13:40
@z4y4ts z4y4ts merged commit 9379edd into dev Mar 30, 2017
@z4y4ts z4y4ts deleted the fix/editor-tabs branch March 30, 2017 13:56
z4y4ts added a commit that referenced this pull request Mar 31, 2017
* dev:
  Remove TinyMCE related code from studio-edit.js (#142)
  Fix few frontend bugs slipped in after code restructuring (#184)
  Update 0.6.3 details in CHANGELOG.md
  Extend Brightcove url regex to include additional set of video urls (#182)
  Refactoring/improve js (#162)
  Switch to master branch of raccoongang/videojs-wistia (#180)
  Add 0.6.1 release to CHANGELOG.md and fix typo in README.md (#179)
  Change make deps-test dependency for faster CI builds (#177)
  Move all external JS and CSS files into video_xblock/static/vendor dir (#176)
  Cover html5 backend with unit tests (#167)
  Add SettingsMixin and default values population from config files (#175)
  Fixes after code review (#174)
z4y4ts added a commit that referenced this pull request Apr 15, 2017
* dev:
  Remove TinyMCE related code from studio-edit.js (#142)
  Fix few frontend bugs slipped in after code restructuring (#184)
  Update 0.6.3 details in CHANGELOG.md
  Extend Brightcove url regex to include additional set of video urls (#182)
  Refactoring/improve js (#162)
  Switch to master branch of raccoongang/videojs-wistia (#180)
  Add 0.6.1 release to CHANGELOG.md and fix typo in README.md (#179)
  Change make deps-test dependency for faster CI builds (#177)
  Move all external JS and CSS files into video_xblock/static/vendor dir (#176)
  Cover html5 backend with unit tests (#167)
  Add SettingsMixin and default values population from config files (#175)
  Fixes after code review (#174)
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