appsembler / xblock-video

Universal Video XBlock with pluggable backend for many video hosting providers
GNU General Public License v3.0
18 stars 50 forks source link

Fix Brightcove blank account id warning #37

Closed bryanlandia closed 3 years ago

bryanlandia commented 3 years ago

Change description

Fix bug where a default value for Brightcove account_id field (either default from field, or as set by SettingsMixin). Correct default will display in Studio edit, and get rid of warning on first save after entering a Brightcove URL (assuming an Account Id defined in settings).

Draft for now, will wait on merge of #35 and rebase.

Type of change

Related issues

https://trello.com/c/AJqLIrFz/449-edx-video-issue-edited-account-id-and-player-id-not-properly-saved-in-new-video-xblock

Checklists

Development

Security

Code review

bryanlandia commented 3 years ago

Closed in favor of everything together in #38