At-mentions change to links, and small change to README

[samcal]

at-mentions will change to a link to '/news/user/<username/' when posting a comment. i also want to change the order of the summary in the readme

[treygriffith]

Why is the change done at the time that the comment is posting rather than before displaying, or even client-side?

Thanks for the PR!

[megamattron]

Hi Sam, thanks for the pull request. I think I agree with Trey though that I'd rather not store the links in the actual comments (since the URL to the user profile might change in the future, etc). So maybe we can use your regex to modify the comment text right before posting? Either way we'll get you into the user list, I think it probably just makes sense to change the point at which we apply the regex. Thanks!

[swelham]

I agree with the above, it should work the same as how the issue numbers are handled - see here

[treygriffith]

I agree with @swelham, issue parsing is a good model to follow. We should probably start modularizing so we have our own customized markdown parser that's specific to Pullup with issues, mentions, etc.

Just to keep track, if implemented this PR should close #292.

[samcal]

Understood. I'll make those adjustments.

[josephwegner]

Thanks for the PR @samcal!

I'm not a man of authority here, but I bet if you put in a PR adding yourself to the authorized users list, it would get merged. You've proved your chops with this PR, regardless of the decision to go a different direction.

Also, if you need any help with the revisions or need to pass them off, feel free to let us know. I know having a PR turned down can feel discouraging at times - that's certainly not our intention.

[megamattron]

Closing since this is covered in #318 - @samcal you should have access shortly.