Next

Testing

Empty comments box

• Create a new post in the newsfeed and copy the URL
• Log out and paste the URL
• A Log in to comment link should show up
• Click on LOGIN and enter creds
• Page will be refreshed and comment composer should show up

Threaded comments box

• (using the post from the previous testing case)
• Add a comment
• Reply to that same comment to create a thread
• Log out and paste the URL
• A Log in to comment link should show up on the root level
• Nothing should show up on the thread
• Click on LOGIN and enter creds
• Page will be refreshed and comment composer should show up on both root and thread levels

Group conversation

• Create (or go to) a public group, copy the URL
• Logout and paste the URL
• A Log in to send a message link should show up
• Click on LOGIN and enter creds
• Page will be refreshed and message composer should show up

Issues

• Closes #2052 (closed).

added scoped label

added scoped label

approved this merge request

assigned to @ottman, @jotto141, and @xander-miller

unapproved this merge request

@edgebal When clicking from a modal, the login modal shows behind the media modal.

added 21 commits

• 34c3727b...b8faccdb - 19 commits from branch master
• f43130b5 - Merge remote-tracking branch 'origin/master' into goal/log-in-notice-comment-thread
• 266bb434 - (fix): Modal z-index on minds.com; stop click bubbling

Compare with previous version

approved this merge request

Thanks for the concise testing instructions @edgebal very useful.

approved this merge request

merged

mentioned in commit 9cb3a5b1

where do i test this?