Modify

Ticket #470 (closed task: to merge)

Opened 3 years ago

Last modified 3 years ago

Enable reviewers to retranscode videos

Reported by: and Owned by: dave
Priority: major Milestone: 4.3 Beta1
Component: AuthorProfile Severity:
Keywords: Cc:
Who will test this: And

Description


Attachments

Change History

comment:1 Changed 3 years ago by anna

  • Milestone changed from 4.1 Beta 1 to 4.1 Beta 2

comment:2 Changed 3 years ago by anna

  • Owner set to dave

comment:3 Changed 3 years ago by dave

This is done very simply, I note the original coder marked out the section to possibly be moved to a configlet, have done this in branch/dave and pulled down to the testing server

comment:4 Changed 3 years ago by dave

  • Status changed from new to closed
  • Resolution set to ready for testing

comment:5 Changed 3 years ago by anna

  • Status changed from closed to reopened
  • Resolution ready for testing deleted

Getting an error after clicking reTranscode & page reloads...

 http://testing.plumi.org:16080/Plone/acl_users/credentials_cookie_auth/require_login?came_from=http%3A//testing.plumi.org%3A16080/Plone/Members/test/videos/test-video/reTranscode

Insufficient Privileges You do not have sufficient privileges to view this page. If you believe you are receiving this message in error, contact the site administration.

comment:6 Changed 3 years ago by dave

  • Status changed from reopened to closed
  • Resolution set to ready for testing

Added this as a custom permission to plumi.content (Manager, Editor and Reviewers have the permission atm)

comment:7 Changed 3 years ago by anna

This is now available.

Just wanted to check before ticking off - the page reports that "The transcoding has not started." though it obviously has. I'm *presuming* this has nothing to do with the changes that have been made in this ticket, and that it's either an anomaly due to the testing site, or an issue with the transcoder. Would you agree?

If so then I'll tick this off as completed.

comment:8 Changed 3 years ago by anna

  • Status changed from closed to reopened
  • Resolution ready for testing deleted

comment:9 Changed 3 years ago by dave

Hrm will have to investigate how it relates in the code

comment:10 Changed 3 years ago by anna

  • Status changed from reopened to closed
  • Resolution set to to merge

Dave checked this on a production instance and it reported correctly (assuming the reporting issues were due to config of the transcoder on the dev site he'd set up).

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.