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.

Sign up for GitHub

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

Jump to bottom

Optionally installing Unoconv during initial deployment #21

Open
SorraTheOrc opened this issue Jan 29, 2018 · 4 comments
Open

Optionally installing Unoconv during initial deployment #21

SorraTheOrc opened this issue Jan 29, 2018 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers Priority 1 (may address) Pull Requests very welcome, but we are unlikely to get to this ourselves.
Milestone
v1.2

Comments

@SorraTheOrc
Copy link
Contributor

Unoconv is a command line program that is used to convert between different office document file formats. It uses an instance of LibreOffice to do the conversion and is used by the Assignment activity to convert documents to pdf so that they can be annotated. If unoconv is not installed - the only impact is that the assignment activities will only allow annotations when students upload a pdf document.

How widely used is this? What is the impact if we install it? Can we make it an optional install?
@SorraTheOrc SorraTheOrc added enhancement New feature or request good first issue Good for newcomers Priority 1 (may address) Pull Requests very welcome, but we are unlikely to get to this ourselves. labels Jan 29, 2018
@jgupta
Copy link

jgupta commented Jan 30, 2018

MoodleCloud specifically mentions this feature in their paid plans as "Document converter". I think it is an important feature.

It will mostly impact Cron and CPU usage during document conversion.

@SorraTheOrc
Copy link
Contributor Author

Given the impact on Cron and CPU I think it is clear that this should be added as an option that is off by default. This is relatively easy to do, but we currently do not have a document on how to do this. Happy to guide folks here and merge pull requests.

@SorraTheOrc
Copy link
Contributor Author

I've been doing some user engagements on this, it would certainly seem like it is common enough to justify adding it as an option. Increasing the priority.

@SorraTheOrc SorraTheOrc changed the title Consider installing Unoconv by default (or optionally) Optionally installing Unoconv during initial deployment Feb 1, 2018
@SorraTheOrc SorraTheOrc added Priority 0 (will address) Pull requests welcome, failing that we will get to this ASAP. and removed Priority 1 (may address) Pull Requests very welcome, but we are unlikely to get to this ourselves. labels Feb 1, 2018
@SorraTheOrc SorraTheOrc added this to the v1.2 milestone Feb 2, 2018
@SorraTheOrc SorraTheOrc added Priority 1 (may address) Pull Requests very welcome, but we are unlikely to get to this ourselves. and removed Priority 0 (will address) Pull requests welcome, failing that we will get to this ASAP. labels May 8, 2018
@SorraTheOrc
Copy link
Contributor Author

Dropping to P1, this is a very clunky solution (requires headless Libre Office or Open Office. Might be better to provide a solution using O365 if possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers Priority 1 (may address) Pull Requests very welcome, but we are unlikely to get to this ourselves.
Projects
None yet
Milestone
v1.2
Development

No branches or pull requests
2 participants
@SorraTheOrc @jgupta