Opened 13 years ago
Last modified 13 years ago
#250 closed defect (FIXED)
Comment links uses media._id instead of media.slug in the URL
Reported by: | joar | Owned by: | Larisa Hoffenbecker |
---|---|---|---|
Priority: | minor | Milestone: | 0.2.0 |
Component: | programming | Keywords: | |
Cc: | Parent Tickets: |
Description
It should be changed so that comment URLs are :: /u/{username}/m/{slug}/c/{comment_id}/#comment instead of /u/{username}/m/{media_id}/c/{comment_id}/#comment
Change History (8)
comment:2 by , 13 years ago
I noticed the edit\_media view does this too; `http://mg.jwandborg.pagekite.me/u/joar/m/4e6a69d5d9d7837042000004/edit/ <http://mg.jwandborg.pagekite.me/u/joar/m/4e6a69d5d9d7837042000004/edit/>`_ (Although you can't access it).
comment:3 by , 13 years ago
Same thing with the add\_attachment view. Those should also use slugs in stead of IDs. Ping me on IRC/update this issue if there's any questions.
comment:4 by , 13 years ago
Milestone: | 0.1.0 → 0.2.0 |
---|
comment:4 by , 13 years ago
Are you still working on this bug, Larisa? If not, I would like to resolve this bug. Thank you!
comment:5 by , 13 years ago
Hey Jakob Kramer, I appreciate you jumping on this :) Larisa said she finished this task but she's new to git, could give me the diff now, but said otherwise she'll try to push it up to the repository this weekend. I encouraged her to try to push it up to git this weekend because I think it's good for contributors to learn how to use git properly. If you feel strongly that this should be done sooner, I can ask for a patch. Otherwise, let's just wait till the weekend!
comment:6 by , 13 years ago
Status: | New → Closed |
---|
Larisa pushed a branch: [https://gitorious.org/\ :sub:`lorochka85/mediagoblin/lorochka85s-mediagoblin](https://gitorious.org/`\ lorochka85/mediagoblin/lorochka85s-mediagoblin) And I just merged it. Just in time for 0.2.0! Thanks Larisa!
Note:
See TracTickets
for help on using tickets.