-
Notifications
You must be signed in to change notification settings - Fork 8
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
Deliver sync video #3
base: deliver_sync_video
Are you sure you want to change the base?
Deliver sync video #3
Conversation
We use MD5 to keep track of the screenshots We delete and re-upload files that had been changed We skip re-upload of files that match md5 and order We delete screenshots if none present locally We don't save if no changes made
We use md5 to keep track of changes in icons Re-upload icon only when changed, or md5 not present in file name on ITC
<%# we are guaranteed to have at least one element because of the group_by %> | ||
<h4><%= screenshots[0].formatted_name %></h4> | ||
<div class="app-screenshot-row"> | ||
|
||
<% if trailer %> | ||
<video width="400px" height="300px" controls poster="<%= trailer.preview_image_path %>"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move width/height to CSS and use % instead (see existing app-screenshot
class one)
So here's a first high level review
|
97c1372
to
e4b8e7d
Compare
We also need to update the README ! This part:
This part:
|
e4b8e7d
to
c360292
Compare
<div class="app-screenshots"> | ||
<h2 style="color: blue">Skipped Trailers</h2> | ||
<p> | ||
Options to skip_trailers activated. <i>Deliver</i> will not modify your app trailers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not need to capitalize. This is referencing the CLI tool.
c360292
to
8ba2d3e
Compare
Upload trailers from trailers_path en-US version propagates for all lang versions - ITC works like that removes from ITC if they exist are before uploading new No ffpmeg support No sync trailers yet
8ba2d3e
to
63f0061
Compare
No description provided.