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

[Release] 2.8.5 - Develop to Master #384

Merged
merged 35 commits into from
Apr 22, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
a475b73
Change null to '' for better support with PHP 8.2 and WP 6.2+
jonnynews Dec 11, 2023
5cf2359
Fix: Undefined array key sub_type
MARQAS Jan 26, 2024
89b9ace
Fix: Only variables should be passed by reference
MARQAS Jan 26, 2024
695a3a1
Don't trigger fetch:apiError if request is aborted (#368)
burhandodhy Jan 29, 2024
638dc70
Update code for PHP5 compatibility
MARQAS Jan 30, 2024
4bb58c1
Move playlist notice outside the controls bar and add check for other…
MARQAS Feb 1, 2024
a179a7f
Merge branch 'develop' into fix/php-82
felipeelia Feb 5, 2024
1d8f12a
Merge pull request #365 from jonnynews/fix/php-82
felipeelia Feb 5, 2024
ec6153d
Merge branch 'develop' into fix/354
felipeelia Feb 5, 2024
d196f1d
Fix: Don't open the detailed view if button is disabled (#372)
burhandodhy Feb 7, 2024
b35126f
Fix: Add support for special characters (#373)
burhandodhy Feb 7, 2024
9e68966
Fix: Don't show 'Label added' message if label data is empty (#374)
burhandodhy Feb 13, 2024
2e1106b
Fix: Search box allignment
MARQAS Feb 15, 2024
abd3661
Merge branch 'develop' into fix/354
MARQAS Feb 15, 2024
d63145c
Merge pull request #370 from 10up/fix/354
MARQAS Feb 15, 2024
8b0bb8b
Assign the value to a variable
MARQAS Feb 15, 2024
ccd4006
Feedback added
MARQAS Feb 15, 2024
4e0a24b
PHP5 compatibility
MARQAS Feb 15, 2024
b68fd8b
Merge branch 'develop' into fix/357
MARQAS Feb 15, 2024
bc77f97
Fix lint issues
MARQAS Feb 15, 2024
28f9217
Add feedback
MARQAS Feb 15, 2024
2dc8879
Merge pull request #369 from 10up/fix/357
MARQAS Feb 16, 2024
f429f15
Remove extra warning for Playlist experience
MARQAS Feb 20, 2024
0efb1a3
Fix linting
MARQAS Feb 20, 2024
53b8d0f
Merge branch 'develop' into fix/SIT-5796
MARQAS Feb 20, 2024
289f4cf
Merge branch 'develop' into fix/346
MARQAS Feb 20, 2024
ebbc296
Merge pull request #376 from 10up/fix/SIT-5796
MARQAS Feb 21, 2024
3054350
Merge branch 'develop' into fix/346
MARQAS Feb 21, 2024
16a58aa
Merge pull request #371 from 10up/fix/346
MARQAS Feb 21, 2024
e4cf3cf
Add Select Option for Label (#378)
burhandodhy Mar 4, 2024
263e180
Send API requests only on the required pages (#379)
burhandodhy Mar 7, 2024
edcab2c
Cache the Api response (#380)
burhandodhy Mar 11, 2024
879084c
Update/docs assets (#375)
jeffpaul Mar 20, 2024
3527297
Fix: Update call fail when labels are empty (#381)
burhandodhy Mar 27, 2024
fabd097
Release 2.8.5 (#382)
burhandodhy Apr 22, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,5 @@
node_modules
release
.sass-cache
vendor
vendor
.DS_Store
Binary file modified .wordpress-org/banner-1544x500.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added .wordpress-org/banner-1544x500.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified .wordpress-org/banner-772x250.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added .wordpress-org/banner-772x250.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
24 changes: 22 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,25 @@ All notable changes to this project will be documented in this file, per [the Ke
### Security
-->

## [2.8.5] - 2023-04-22

### Changed
- Add Select field for labels. Props [@burhandodhy](https://github.com/burhandodhy) via [#378](https://github.com/10up/brightcove-video-connect/pull/378).
- Cache the API response and display a message from where user can retry the API again. Props [@burhandodhy](https://github.com/burhandodhy) and [@jonnynews](https://github.com/jonnynews) via [#380](https://github.com/10up/brightcove-video-connect/pull/380).
- Update readme and assets. Props [@jeffpaul](https://github.com/jeffpaul) via [#375](https://github.com/10up/brightcove-video-connect/pull/375).

### Fixed
- First parameter type of add_submenu_page() when no parent slug is sent. Props [@jonnynews](https://github.com/jonnynews) via [#365](https://github.com/10up/brightcove-video-connect/pull/365).
- Display error message when the previous request is aborted. Props [@burhandodhy](https://github.com/burhandodhy) via [#368](https://github.com/10up/brightcove-video-connect/pull/368).
- PHP warning while editing a label. Props [@MARQAS](https://github.com/MARQAS) and [@felipeelia](https://github.com/felipeelia) via [#369](https://github.com/10up/brightcove-video-connect/pull/369).
- PHP warning while saving a video. Props [@MARQAS](https://github.com/MARQAS) and [@felipeelia](https://github.com/felipeelia) via [#370](https://github.com/10up/brightcove-video-connect/pull/370).
- Move playlist notice above controls bar. Props [@MARQAS](https://github.com/MARQAS) and [@felipeelia](https://github.com/felipeelia) via [#371](https://github.com/10up/brightcove-video-connect/pull/371).
- Clicking on the "Edit" and "Preview" buttons for the inactive videos breakes the layout. Props [@burhandodhy](https://github.com/burhandodhy) and [@felipeelia](https://github.com/felipeelia) via [#372](https://github.com/10up/brightcove-video-connect/pull/372).
- Add support for special characters. Props [@burhandodhy](https://github.com/burhandodhy) and [@cr0ybot](https://github.com/cr0ybot) via [#373](https://github.com/10up/brightcove-video-connect/pull/373).
- Search box alignment in Media modal. Props [@MARQAS](https://github.com/MARQAS) and [@felipeelia](https://github.com/felipeelia) via [#376](https://github.com/10up/brightcove-video-connect/pull/376).
- Limit API calls on unwanted admin pages. Props [@burhandodhy](https://github.com/burhandodhy) via [#379](https://github.com/10up/brightcove-video-connect/pull/379).
- Video update API fails when labels are empty. Props [@burhandodhy](https://github.com/burhandodhy) via [#381](https://github.com/10up/brightcove-video-connect/pull/381).

## [2.8.4] - 2023-11-30

### Added
Expand Down Expand Up @@ -99,7 +118,7 @@ All notable changes to this project will be documented in this file, per [the Ke
- Video and Playlist experiences displays wrong block settings. Props [@Rahmon](https://github.com/Rahmon), and [@oscarssanchez](https://github.com/oscarssanchez) via [#250](https://github.com/10up/brightcove-video-connect/pull/250).
- Update attribute type from int to string. Props [@Rahmon](https://github.com/Rahmon), and [@oscarssanchez](https://github.com/oscarssanchez) via [#246](https://github.com/10up/brightcove-video-connect/pull/246).
- Pass sizing attribute to determine if responsiveness should be enabled. Props [@Rahmon](https://github.com/Rahmon), and [@oscarssanchez](https://github.com/oscarssanchez) via [#249](https://github.com/10up/brightcove-video-connect/pull/249).
- Fix broken Brightcove experiences embedding. Props [@Rahmon](https://github.com/Rahmon), and [@oscarssanchez](https://github.com/oscarssanchez) via [#248](https://github.com/10up/brightcove-video-connect/pull/248)
- Fix broken Brightcove experiences embedding. Props [@Rahmon](https://github.com/Rahmon), and [@oscarssanchez](https://github.com/oscarssanchez) via [#248](https://github.com/10up/brightcove-video-connect/pull/248).

## [2.5.0] - 2021-12-21

Expand Down Expand Up @@ -136,7 +155,7 @@ All notable changes to this project will be documented in this file, per [the Ke
### Breaking Changes
- BC_Utility API changed:
- See `set_cache_item` and `delete_cache_item` in `includes/class-bc-utility.php`.
- Removed `remove_deleted_players` function.
- Removed `remove_deleted_players` function.
### Fixed
- Undefined index warnings. Props [@sanketio](https://github.com/sanketio) via [#197](https://github.com/10up/brightcove-video-connect/pull/197).
- Typo for the `$allowedtags` global used in conjunction with wp_kses. Props [@theskinnyghost](https://github.com/theskinnyghost) via [#203](https://github.com/10up/brightcove-video-connect/pull/203).
Expand Down Expand Up @@ -443,6 +462,7 @@ All notable changes to this project will be documented in this file, per [the Ke
- First release

[Unreleased]: https://github.com/10up/brightcove-video-connect/compare/master...develop
[2.8.5]: https://github.com/10up/brightcove-video-connect/compare/2.8.4...2.8.5
[2.8.4]: https://github.com/10up/brightcove-video-connect/compare/2.8.3...2.8.4
[2.8.3]: https://github.com/10up/brightcove-video-connect/compare/2.8.2...2.8.3
[2.8.2]: https://github.com/10up/brightcove-video-connect/compare/2.8.1...2.8.2
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

Are you looking to handle your [Brightcove](https://www.brightcove.com/en/online-video-platform) Video and Playlist library natively from within WordPress?

With this plugin, developed by [10up](http://10up.com), you have the power to handle multiple accounts and video libraries, upload videos and add them to playlists, render shortcodes with your videos and all from within the WordPress admin interface.
With this plugin, you have the power to handle multiple accounts and video libraries, upload videos and add them to playlists, render shortcodes with your videos and all from within the WordPress admin interface.

## Support Notice

Expand Down Expand Up @@ -115,7 +115,7 @@ However, in order for it to work you need to make sure you have multiple audio t

## Support Level

**Active:** 10up is actively working on this, and we expect to continue work for the foreseeable future including keeping tested up to the most recent version of WordPress. Bug reports, feature requests, questions, and pull requests are welcome.
**Active:** Brightcove is actively working on this, and we expect to continue work for the foreseeable future including keeping tested up to the most recent version of WordPress. Bug reports, feature requests, questions, and pull requests are welcome.

## Changelog

Expand Down
35 changes: 32 additions & 3 deletions assets/css/brightcove_video_connect.css
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,7 @@ div.brightcove-uploader div.brightcove-messages .updated {

.brightcove-modal .brightcove .media-toolbar-primary.search-form input[type=search] {
float: left;
margin: 0;
padding-bottom: 0px;
width: 200px; }

Expand Down Expand Up @@ -129,7 +130,9 @@ div.brightcove-uploader div.brightcove-messages .updated {
.media-frame:not(.hide-menu) .media-frame-menu {
width: 300px; } }

.brightcove.media-frame-menu .attachment-detail .playlist-details label.experiencelabel, .brightcove.media-frame-menu .attachment-detail .video-details label.experiencelabel, select.experienceplayer {
.brightcove.media-frame-menu .attachment-detail .playlist-details label.experiencelabel,
.brightcove.media-frame-menu .attachment-detail .video-details label.experiencelabel,
select.experienceplayer {
width: 100%; }

.bc-page-icon {
Expand Down Expand Up @@ -633,15 +636,41 @@ div.brightcove-uploader div.brightcove-messages .updated {
.brightcove.video-edit .caption-secondary-fields label:nth-child(1) {
padding-right: 1em; }
.brightcove.video-edit .caption-secondary-fields label span,
.brightcove.video-edit .caption-secondary-fields label input[type="text"],
.brightcove.video-edit .caption-secondary-fields label input[type='text'],
.brightcove.video-edit .caption-secondary-fields label select {
margin: 0;
min-width: 100%;
text-align: left;
width: 100%; }
.brightcove.video-edit .caption-secondary-fields label input[type="checkbox"] {
.brightcove.video-edit .caption-secondary-fields label input[type='checkbox'] {
margin: auto; }

.brightcove.video-edit .bc-labels-list {
margin: 0;
text-align: left; }
.brightcove.video-edit .bc-labels-list li {
background-color: #fff;
border-radius: 4px;
border: 1px solid #8c8f94;
display: inline-block;
margin: 0 10px 10px 0;
white-space: nowrap; }
.brightcove.video-edit .bc-labels-list button {
background-color: #f6f7f7;
border-bottom-left-radius: 4px;
border-right: 1px solid #8c8f94;
border-top-left-radius: 4px;
border: none;
cursor: pointer;
padding: 10px; }
.brightcove.video-edit .bc-labels-list span {
float: none; }
.brightcove.video-edit .bc-labels-list .label-name {
padding: 10px; }

.brightcove.video-edit .add-labels-container select {
margin-left: 0; }

.playlist-edit.brightcove.attachment-details {
width: 100%;
height: auto;
Expand Down
2 changes: 1 addition & 1 deletion assets/css/brightcove_video_connect.min.css

Large diffs are not rendered by default.

4 changes: 4 additions & 0 deletions assets/css/jquery-ui-datepicker.min.css

Large diffs are not rendered by default.

139 changes: 53 additions & 86 deletions assets/js/brightcove-admin.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion assets/js/brightcove-admin.js.map

Large diffs are not rendered by default.

7 changes: 5 additions & 2 deletions assets/js/brightcove-admin.min.js

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions assets/js/src/models/media-collection.js
Original file line number Diff line number Diff line change
Expand Up @@ -264,14 +264,14 @@ var MediaCollection = Backbone.Collection.extend({
return fallback.sync.apply(this, arguments);
},

fetchFail: function () {
fetchFail: function (response, textStatus) {
if (this.pageNumber > 1) {
this.pageNumber--;
}
wpbc.broadcast.trigger('fetch:finished');
wpbc.broadcast.trigger('spinner:off');
wpbc.broadcast.trigger('fetch:apiError');
if (status === 'abort') {
if (textStatus !== 'abort') {
wpbc.broadcast.trigger('fetch:apiError');
}
},

Expand Down
Loading
Loading