Skip to content

Should be able to disable "2nd class" outputs in admin settings #1154

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

Closed
hwine opened this issue Feb 11, 2015 · 6 comments
Closed

Should be able to disable "2nd class" outputs in admin settings #1154

hwine opened this issue Feb 11, 2015 · 6 comments
Assignees
Labels
Good First Issue Good for new contributors Sprintable Small enough to sprint on

Comments

@hwine
Copy link

hwine commented Feb 11, 2015

At the moment, PDF and ePub formats are "2nd class" outputs of the system, in that the author does not get notified of production failures specific to those formats. (see, e.g. #1121 which impacted users as described in #1120)

Since lossless conversion to the 2nd class formats can't be guaranteed, an author may need to invest additional time to produce source that will convert cleanly. I think that choice should be up to the author, not a (hidden) cost of using read the docs.

Adding checkboxes to enable/disable production of these alternate formats to the administration page would be one way to implement.

@agjohnson
Copy link
Contributor

I agree. I'm not sure of the statistics of the number of users producing pdf/epub output, but i think exposing these as options some how to the user would reduce confusion -- especially since pdf builds can be intermittent problematic in our build evironment.

We're trying to reduce the config options, as the user experience is awful, but this might be a large enough configuration option to warrant a UI element for configuration.

@tisdall
Copy link

tisdall commented May 6, 2015

@agjohnson it seems like pdf/epub are defaulted to on, so wouldn't it be all projects end up producing those? Or am I missing something?

@agjohnson
Copy link
Contributor

@tisdall i suppose i meant using pdf output, this is the number I'm not sure of. You are correct that all projects default with pdf/epub output enabled.

@ericholscher
Copy link
Member

Our builder code knows how build all the types separately. The main thing
we need is a UI for selecting them. A PR with this would definitely be
accepted. It's on our long list of things to do otherwise.

On Wed, May 6, 2015 at 1:57 PM, Anthony [email protected] wrote:

@tisdall https://github.com/tisdall i suppose i meant using pdf
output, this is the number I'm not sure of. You are correct that all
projects default with pdf/epub output enabled.


Reply to this email directly or view it on GitHub
#1154 (comment)
.

Eric Holscher
Maker of the internet residing in Portland, Oregon
http://ericholscher.com

@gregmuellegger
Copy link
Contributor

This is done with the merge of #1344.

@tisdall
Copy link

tisdall commented Jul 10, 2015

@gregmuellegger, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Good for new contributors Sprintable Small enough to sprint on
Projects
None yet
Development

No branches or pull requests

5 participants