jberkel / sms-backup-plus

Backup Android SMS, MMS and call log to Gmail / Gcal / IMAP
https://play.google.com/store/apps/details?id=com.zegoggles.smssync
Apache License 2.0
1.79k stars 497 forks source link

attach file through image hint for readme bug report #1038

Closed cauerego closed 3 years ago

cauerego commented 3 years ago

much better than using gist. it clearly works: https://github.com/jberkel/sms-backup-plus/issues/967#issuecomment-699924157

kurahaupo commented 3 years ago

Images are a lot easier for the person submitting, so if it's a case of "gist is too hard", then an image is preferable to nothing.

But if the log runs to more than a screenful - either by length or width - then information will be lost, because by the time we want to ask "what was the bit just off the bottom of the screen?", the log has already gone.

So let's say "we would prefer the log as text, but if you're not familiar with using gist then you're welcome to attach a screen shot instead"

cauerego commented 3 years ago

i meant we can use the image button on github to attach the log file, which is text or whatever and not a screenshot. it's what i've done there!

kurahaupo commented 3 years ago

ooooh, right, that makes more sense.

kurahaupo commented 3 years ago

Perhaps it should be called the "attach file feature" so that people are less likely to assume that we want images uploaded?

cauerego commented 3 years ago

Perhaps it should be called the "attach file feature" so that people are less likely to assume that we want images uploaded?

yes, but i'm not doing a pull request for that! 🤣

kurahaupo commented 3 years ago

@cauerego amended and merged.