Opened 11 years ago
Last modified 7 years ago
#364 accepted enhancement
Shotwell integration with mediagoblin API
|Reported by:||Christopher Allan Webber||Owned by:|
Description (last modified by )
This is obviously somewhat preliminary, as we don't have an API quite yet as I'm writing this, but... We certainly want to integrate with all sorts of devices, but Shotwell would be a fantastic starting point as they're FaiF, come in near every gnome-oriented distro preinstalled, have similar goals, and have expressed interest in bundling a mediagoblin plugin. `http://redmine.yorba.org/projects/shotwell/wiki/ShotwellArchWritingPlugins <http://redmine.yorba.org/projects/shotwell/wiki/ShotwellArchWritingPlugins>`_
Change History (7)
comment:1 by , 11 years ago
comment:2 by , 11 years ago
|Type:||defect → enhancement|
comment:3 by , 10 years ago
|Status:||accepted → closed|
I am very waffley about whether this should be on our bug tracker. Cc'ing joar in case he has opinions. We do have the API now. But we also closed out an automgtic related bug because it wasn't directly related to MediaGoblin.
I'm going to leave it open but if nobody's done anything with this in the next couple of months, it should be closed.
comment:4 by , 10 years ago
|Status:||closed → reopened|
Not sure why I marked it fixed with that statement above. :)
comment:5 by , 10 years ago
|Status:||reopened → review|
Since Elrond's piwigo plugin we can post from shotwell to mediagoblin using this patch to actually upload the submitted files: https://gist.github.com/joar/5601717
comment:6 by , 8 years ago
|Status:||review → in_progress|
Joar, I'm just looking at this now unfortunately... anyway, it looks like this actually ought to be using the new submit tools if we're going to make use of it in
mediagoblin.submit.lib:submit_media(). We could avoid code duplication if we do this.
I'm going to pass this back to you, though I'm also a bit lost since I haven't used the piwigo plugin. As long as someone who *does* use the Piwigo plugin tests and verifies that it works, and the code looks clean, I'd be willing to merge this to master.
comment:7 by , 7 years ago
|Status:||in_progress → accepted|
The original url for this bug was http://bugs.foocorp.net/issues/717 .