ariana-paris / otwarchive-gcode-issues

Apache License 2.0
0 stars 0 forks source link

Commenting should all take place within a single page #31

Closed ariana-paris closed 9 years ago

ariana-paris commented 9 years ago

Original issue 31 created by ariana-paris on 2008-04-21T21:04:48.000Z:

To reproduce:

  1. Find a story
  2. Post a comment

You will end up on a page with just the comment and edit/back links on it.

Instead, comment-posting should all happen within the page for the topmost object being commented-on. Posting a comment should make it appear within that page.

ariana-paris commented 9 years ago

Comment #1 originally posted by ariana-paris on 2008-04-25T18:40:36.000Z:

Started, and I'm going to try to add more Javascript so that instead of submitting and getting redirected to the same page, you stay on the page as the changes are added to it

ariana-paris commented 9 years ago

Comment #2 originally posted by ariana-paris on 2008-05-07T11:30:56.000Z:

Elz, just wanted to let you know that I'm currently going through and refactoring the act-as-commentable plugin.

ariana-paris commented 9 years ago

Comment #3 originally posted by ariana-paris on 2008-05-27T01:44:32.000Z:

Added to this in r232 - you can add and edit comments via Javascript now, and I added live validation to comments as of r241.

ariana-paris commented 9 years ago

Comment #4 originally posted by ariana-paris on 2008-06-02T22:17:44.000Z:

Seems to be working well!

ariana-paris commented 9 years ago

Comment #5 originally posted by ariana-paris on 2009-02-25T01:43:18.000Z:

<empty>

ariana-paris commented 9 years ago

Comment #6 originally posted by ariana-paris on 2009-02-25T03:23:29.000Z:

Updating status on discussion with Jennifew and Sidra

ariana-paris commented 9 years ago

Comment #7 originally posted by ariana-paris on 2010-11-13T22:14:24.000Z:

listing all former issues for automated tests

ariana-paris commented 9 years ago

Comment #8 originally posted by ariana-paris on 2010-11-14T19:40:54.000Z:

Converting from NeedsTest status to NeedsTest label