-
Notifications
You must be signed in to change notification settings - Fork 51
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
Avalon 7.1 #4052
Merged
Avalon 7.1 #4052
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Allow intercom response to take an hour before timing out.
Allow for more than 100 collections
Avalon Media System, not The Avalon Media System
Fix ActionMailer not using SES
Remove Article
Fix timeliner config missing in development
Fix and standardize styling for all tables
SME build with Peaks.js upgrade
- Add master file ids - Change derivative ids to be object id instead of track id which is already present at track_id - Allow inlining of structure in media object show request by include_structure query paramter Co-authored-by: Dananji Withana <dwithana@iu.edu>
SME build - validation logic fix
Use latest SME
This line was removed from avalon 7 as part of the transcoding backend switch to newer active_encode with ffmpeg adapter instead of matterhorn. It is reinserted here to avoid breaking streaming of intercom pushed items that are not managed streams. In this case hls_url isn't being set and location_url is the rtmp url because setStreamingLocations isn't setting it because the derivative is unmanaged. In the end the item is not streamable since it doesn't have an HLS url and only an RTMP url. It would be good to switch to get rid of the need for two separate fields for RTMP and HLS streaming urls but it would require a larger refactoring of the code and possibly a data migration which probably isn't worth it at this stage.
Save hls_url if present
The previous version needlessly required that all guard clauses run even when a prior line had matched and initialized mf_title. Switching to explicit if/elsif allows for early breaking. This switch helps speed up the view page of an item with hundreds of sections by avoiding the third case of file_location which turns out to be expensive.
Use if/elsif instead of ||= with guard clause
Replace ImageMagick with FastImage
Bug fix: Media player disappearing on reload in Edge 16
Pushing upstream an IU Customization
Fix ActionMailer not using smtp configs
Upgrade rails-html-sanitizer to deal with loofah deprecation warnings
Show errors even if push was successful.
Add rake task to clean out old sessions Schedule session cleanup task to run every 6 hours Push upstream IU customization
Lower quality scale on JPEG creation
Don't force encoding to 30fps
Increase Default Poster Size
Bump version to prepare for release
Avalon 7.1.0RC1
Remove identifiers
Do not inline structure if query parameter is false
phuongdh
approved these changes
Mar 17, 2020
joncameron
approved these changes
Mar 17, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.