Closed (fixed)
Project:
Activity Stream
Version:
6.x-1.0-beta2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
17 Feb 2009 at 09:03 UTC
Updated:
19 Sep 2009 at 01:18 UTC
Jump to comment: Most recent file
Comments
Comment #1
toddejohnson commentedI found a better icon to be included just replace identica.png with the attached file.
Comment #2
akalsey commentedThat's fantastic. Can you create a Drupal.org project for it? See http://drupal.org/node/386992
Comment #3
akalsey commentedComment #4
akalsey commentedAny progress on creating a project for this?
Comment #5
papapep commentedMoved to a more appropiate place.
Comment #6
akalsey commentedPlease add your own item to the issue queue insteadof hijacking an existing issue with an unrelated question.
Comment #7
papapep commentedI beg your pardon. I really didn't notice this was an specific issue thread. I thought it was a general identi.ca/activity stream thread.
Comment #8
butler360 commentedI've made a very small improvement.
It used to be on line 49:
$name . l($node->title, $activity->link)I've changed it to this:
$name . " " . l($node->title, $activity->link)This adds a space between the username and the post from Identi.ca, which looks much better.
So intead of "Johnjohndoe: This is my message!" you get:
"John johndoe: This is my message!"
I don't know how to create patches, so I'm posting it this way.
Lastly, is there a way to trim the username from the post? I don't like seeing [Drupalusername] [Identi.ca username]: [Identi.ca post]. I'd prefer to remove the Identi.ca username. I know that this module uses the Identi.ca RSS feed, though, which includes your username in the title/message. Is there some way to maybe trim everything before the first colon?
Comment #9
forestmars commented"Patch Attached."
(~forest mars)
Comment #10
forestmars commented(status changed to need review for patch queue.)
Comment #11
toddejohnson commentedI have created a proper project for the module at http://drupal.org/project/activitystream_identica. It includes the patch. I will post a separate issue for getting the link fixed.