Needs work
Project:
Twitter
Version:
7.x-6.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
27 Nov 2009 at 19:54 UTC
Updated:
5 Aug 2015 at 11:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sirkitree commented++ this would be some sweet functionality!
Comment #2
benklocek commented+1 subscribe
Comment #3
derhasi commentedI added a patch for implementing storage of longitude and latitude of the tweet. Additionaly there are views fields for both values, that can be used by Openlayers & Co. This should fix the title Store tweet location.
Removing a location from twitter.com would be a new issue for the API, I guess. Implemenation with location's or geo's storage options should be an issue for a contrib module.
It would be nice someone could review the patch.
P.S.: sorry for the whitespace changes, but as it is part of the Drupal coding standards I configured my IDE to convert it automatically.
Comment #4
pribeh commentedsubscribe
Comment #5
derhasi commentedpribeh, did you test the attached patch of #3?
Comment #6
pribeh commentedHi darhasi. I will get to testing this today and report back. I'm just setting things up on a dev site to test this.
Comment #7
pribeh commentedTesting of this patch will have to wait until tomorrow. Thanks derhasi for putting your efforts into this. I do plan on using this on a live site if this works out.
Comment #8
greg.harveyWow, just looking in to this for a client. Good to find the beginnings of a patch. I'll need to do what Richard needed to do, so if we follow this through with our customer we'll be able to test this and hopefully get it finished.
For now, subscribe and +1!
Comment #9
mrf commentedStill seems like a good idea to me, but should be done on 7.x-5.x-dev first.
Comment #10
jasonflaherty commentedJust following up with this request. Was it implemented in 7.x-5.x-dev or into the latest module?
Comment #11
damienmckenna@jasonflaherty: It hasn't been added to any branch yet.
Comment #12
damienmckennaComment #13
damienmckenna