enhancement
avatar wilsonge
wilsonge
14 Feb 2016

At the moment we ask for relevent server information. Since 3.5 we now have an export feature (
image ) can we add support for just uploading that config into jissues please?

avatar wilsonge wilsonge - open - 14 Feb 2016
avatar b2z
b2z - comment - 18 Feb 2016

I think so. Should we support other file extensions like pdf, docx etc?
https://github.com/blog/2061-attach-files-to-comments

avatar wilsonge
wilsonge - comment - 18 Feb 2016

It depends how you wanna render them. pdf.js is Apache license which is only compatible with GPLv3 for example... :(

avatar brianteeman
brianteeman - comment - 18 Feb 2016

That's not necessarily a problem as long as it is used on jissues and not
in the cms itself
On 18 Feb 2016 9:21 am, "George Wilson" notifications@github.com wrote:

It depends how you wanna render them. pdf.js is only GPLv3 and higher for
example... :(


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

avatar wilsonge
wilsonge - comment - 18 Feb 2016

Edited comment above for clarity - although I think Brian's point still stands...

avatar b2z b2z - assigned - 18 Feb 16
avatar b2z b2z - change - 18 Feb 2016
Labels Added: enhancement
avatar mbabker
mbabker - comment - 18 Feb 2016

We don't have to support rendering them though. Just use "normal" browser behaviors for text file types like that (link it and let the user/browser decide to open in browser, download file, etc.). With images there's an expectation to render them inline, files not so much.

avatar wilsonge
wilsonge - comment - 18 Feb 2016

Fine. Although doing some kind of export treatment for the Joomla server info would be nice - so people can see it as part of the issue 'automagically'

avatar mbabker
mbabker - comment - 18 Feb 2016

If you really wanted to inline the data you could tell people to upload the JSON file and our code could read that and spit out "human friendly" text. But honestly I don't see it being mandatory to do that as long as the data gets uploaded/attached to an issue in any form.

avatar wilsonge
wilsonge - comment - 18 Feb 2016

I don't think it should be mandatory. But it be an option.... I think there's a decent number of bug reports where users don't know what server info to upload

avatar mbabker
mbabker - comment - 18 Feb 2016

To an extent, I agree. But ultimately, the tracker's comment fields shouldn't be doing too much beyond what GitHub offers (all our extra functionality is around features GitHub doesn't have). This borderline crosses into "we're doing something that the user cannot do on GitHub" if we start manipulating comment text based on file uploads matching certain criteria. Support file uploads the same way GitHub does, +1 all day every day. Magically take a JSON or TXT file exported from Joomla, uploaded through the regular file processor, and spit it into the user's input field? I wouldn't encourage it.

avatar brianteeman
brianteeman - comment - 18 Feb 2016

Def more useful for the forum than for the tracker
On 18 Feb 2016 6:35 pm, "Michael Babker" notifications@github.com wrote:

To an extent, I agree. But ultimately, the tracker's comment fields
shouldn't be doing too much beyond what GitHub offers (all our extra
functionality is around features GitHub doesn't have). This borderline
crosses into "we're doing something that the user cannot do on GitHub" if
we start manipulating comment text based on file uploads matching certain
criteria. Support file uploads the same way GitHub does, +1 all day every
day. Magically take a JSON or TXT file exported from Joomla, uploaded
through the regular file processor, and spit it into the user's input
field? I wouldn't encourage it.


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

avatar b2z
b2z - comment - 20 Feb 2016

Please test #747

avatar mbabker mbabker - change - 10 Mar 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-03-10 02:39:22
Closed_By mbabker
avatar mbabker mbabker - close - 10 Mar 2016

Add a Comment

Login with GitHub to post a comment