<div dir="ltr">looking today.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 27, 2017 at 4:54 PM, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Sep 27, 2017 at 01:50:32PM +0200, Tomaz Canabrava wrote:<br>
> On Wed, Sep 27, 2017 at 5:41 AM, Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br>
><br>
> > On Tue, Sep 26, 2017 at 11:21:59PM -0400, Federico Masias wrote:<br>
> > > ><br>
> > > ><br>
> > > > Yes it is. Please do.<br>
> > > ><br>
> > ><br>
> > > Done.<br>
> ><br>
> > Thanks, already merged<br>
> ><br>
> > > > > Facebook upload mostly works, but Subsurface asks for an album name<br>
> > to<br>
> > > > place the photo into an album... but Facebook simply posts it to an<br>
> > album<br>
> > > > called "Subsurface Photos" -- no mention of the user defined album<br>
> > > > anywhere. I'd also caution against using the verbiage of "profile<br>
> > picture"<br>
> > > > in the instructions when referring to the dive profile graph, as that<br>
> > > > phrase means something else completely to Facebook users.<br>
> > > ><br>
> > > > Both great points. Can you fix this in the code? Otherwise, please<br>
> > raise<br>
> > > > an issue on Github.<br>
> > > > These are both easy to address<br>
> > > ><br>
> > > ><br>
> > > I was able to change the string, but I've noticed a few more things about<br>
> > > the parameters that are passed to Facebook, so I'll raise a Github issue<br>
> > > about that shortly, but the short version is that I'm not sure if we're<br>
> > > passing the wrong info to the FB API, or it's being ignored -- both for<br>
> > the<br>
> > > album name and privacy settings. I don't have time tonight to<br>
> > investigate,<br>
> > > and I'm sure someone around here is much better at it than I -- but if<br>
> > it's<br>
> > > not fixed by someone else, I'll try and follow up on it when I do have a<br>
> > > bit more time.<br>
> ><br>
> > Tomaz recently updated our code to the latest API version. Did you test<br>
> > with the latest test version or with the last release version? If the<br>
> > former, let's make sure you tag @tcanabrava in the issue so he can look<br>
> > into this to see if maybe something got mixed up with the change to the<br>
> > newer API.<br>
> ><br>
><br>
> The latest api version was just a number change, no actuall api methods<br>
> where modified.<br>
<br>
</div></div>Did you have a chance to take a look at the bug report?<br>
<br>
<a href="https://github.com/Subsurface-divelog/subsurface/issues/612" rel="noreferrer" target="_blank">https://github.com/Subsurface-<wbr>divelog/subsurface/issues/612</a><br>
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888"><br>
/D<br>
</font></span></blockquote></div><br></div>