dev-launchers-platform icon indicating copy to clipboard operation
dev-launchers-platform copied to clipboard

[US - DEV] Link Users to Comments with Database Relation

Open dbradham opened this issue 1 year ago • 2 comments

Currently, the only reference of an author on a comment object is their name as plain text. This is a problem for a number of reasons.

  1. If two users have the same name we cannot differentiate who submitted the comment.
  2. If we want to send a notification when someone replies to a given user's comment, we need to query existing users based on their name
  3. If a user ever changes their name, the relationship "breaks"

Acceptance Criteria

Establish a database relationship between Comments and Users similar to Ideas and Users

dbradham avatar Feb 18 '24 21:02 dbradham

Hey @RETprojects, if you're on this task right now, can you please update its status and move it to the "work in progress" column? Thanks!

JulieMass avatar Mar 06 '24 12:03 JulieMass

For history documentation: As discussed today with @RETprojects and @dbradham in Discord, we are moving this ticket to the icebox so Remi can focus on his current bug #953. We’ll come back to this one later. Thank you Remi!

JulieMass avatar Apr 28 '24 23:04 JulieMass

Everything involving the display names seems to be working properly in staging, except when I go to a workshopping page with comments that were posted before we reworked the storage of users' names on comments, I get a 404 Page Not Found error. I encountered these comments in localhost, and they don't have the proper information stored in them. I believe that if these earlier comments were deleted or possibly edited to reflect the current comment structure, then this 404 error would be resolved and new comments could be posted on these pages.

RETprojects avatar Jun 30 '24 23:06 RETprojects

Probably should provide solution that addresses bad data, but we are still in BETA, so Meh!

chadcrotchett avatar Jul 14 '24 22:07 chadcrotchett