Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Display @ before mentions #2

Merged
merged 1 commit into from Feb 11, 2017
Merged

Display @ before mentions #2

merged 1 commit into from Feb 11, 2017

Conversation

SuperTux88
Copy link
Member

This adds the @ before the mentions link as suggested in diaspora/diaspora#7269.

After this is released I can add the same to the diaspora backend.

(also fixed the last test in base.txt)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d70fd61 on SuperTux88:at-before-mention-links into 03c3cb2 on diaspora:master.

@SuperTux88 SuperTux88 mentioned this pull request Feb 10, 2017
Closed
svbergerem added a commit that referenced this pull request Feb 11, 2017
@svbergerem
Merge pull request #2 from SuperTux88/at-before-mention-links
ae2a9f9 
Display @ before mentions
@svbergerem svbergerem merged commit d70fd61 into diaspora:master Feb 11, 2017
@svbergerem svbergerem removed their request for review February 11, 2017 02:03
@svbergerem
Copy link
Member

Thank you!

@SuperTux88 SuperTux88 deleted the at-before-mention-links branch February 11, 2017 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@SuperTux88 @coveralls @svbergerem