[FEAT] Record the user that initiated a notification event

[chungthuang]

Summary
As the owner of an idea card, I'd like to know who commented on my ideas.

Describe the solution you'd like
Add a new field in the event collection to record the user that triggered the event. We need to update the lifecycle method that generates the event to send the user information.

Acceptance criteria
Integration test should assert that when user A comments on an idea card owned by user B, user B is able to see user A commented on the idea. For privacy concerns, user B should only see public information about user A.

[j-peace]

1st scenario: Other users comment in my Idea and my notifications are on:

• WHEN: I create an Idea.
• AND: Other users comments in my idea.
• THAN: I should receive a notification (with just public information who comments in my idea).

2nd scenario: Other users comment in my Idea and my notifications are off:

• WHEN: I create an Idea.
• AND: Other users comments in my idea.
• THAN: I should not receive a notification.

3rd scenario: I comment in my own Idea:

• WHEN: I create an Idea.
• AND: I comment in my idea.
• THAN: I should not receive a notification.

Important to validate

• What is the expected default settingd? Comment notificarions on or off ?
• Users that has more than one idea, should be able to turn on/off comment notifications for junt an specific idea? Or the settings are valid for all their ideas?

[dbradham]

@tiagopazhs @oriyomibadmus do either of you have availability to discuss this on Tuesday after 6:30pm Central time?

[j-peace]

Hey @dbradham 6:30pm CT looks good ?

[j-peace]

Sorry for the latest answear, we can schedulle another time ?

[dbradham]

• Users that has more than one idea, should be able to turn on/off comment notifications for junt an specific idea?

After discussion, we have determined scenario 2 & 3 are outside the acceptance criteria for this ticket.

The expected settings are for notifications to be on.

@tiagopazhs has asked the IdeaSpace team for feedback on the idea-specific notification settings.

[j-peace]

@oriyomibadmus as we discussed with @dbradham the field Comment.author is not used anymore. Please remove that.

[j-peace]

@oriyomibadmus and @dbradham,

We can have a discussion here why we weren't able to reproduce the error using the admin pannel ?

• params used to create the comment:
  

• note that when we retrieve the comment after it was created the field user was not persisted:
  

[dbradham]

This behavior is unstable due to the hook on line 6 of this screenshot @tiagopazhs

• We are not authenticated from the dashboard as a user, so the application appears to default to the first user created in the database.
• We should test the user relation in an environment where we are authenticated.
• For example, we can run the frontend application, authenticate with the backend, and then submit a comment.
• Alternatively, we could authenticate with postman or a test script and then submit a comment payload with a token to the application.

[dbradham]

Cross testing:
I can see the event is linked to my user. (id: 48)

I can see my user is linked to the comment, correctly, because I am logged into the application. This is different from Tiago's test where he was not authenticated and using the Strapi admin panel-

[dbradham]

@oriyomibadmus The title of the Event seems strange "null commented on...".

It seems that this was working differently for Vivien 11 days ago.

I believe the issue is because my user has no username-

When I updated my username in the staging database, I got the expected result:

@chungthuang please let us know if the acceptance criteria is met here. It seems to work as expected from my tests.

[dbradham]

@chungthuang I am moving this to Done per our discussion in the team sync call