feat: Implement automatic spam detection banners #8

santhosh-apphelix-2u · 2025-11-07T12:25:59Z

This PR introduces the Auto-Flag Banner feature as part of the AI Moderation UI implementation.
The banner is automatically displayed when content is flagged as possible spam, based on the is_spam flag returned from the API response.

Key Updates

Added Auto-Flag Banner component for thread, response, and comment levels.
Banner message: "Content automatically reported as possible spam pending staff review."
Applied consistent styling aligned with the existing “Content Reported for Staff Review” banner.
Integrated banner visibility logic using the is_spam key from API response.
Added an **information ** icon at the end of the banner:
On click, it opens a modal explaining the purpose of the automatic spam detection.

Ref Ticket - https://2u-internal.atlassian.net/browse/COSMO2-771

fix: resolve InContext Sidebar post menu dropdown clipping that triggered scroll

Added AutoSpamAlertBanner component to display spam alerts.

Added AutoSpamAlertBanner to Post component and adjusted spam flag logic.

Copilot

Pull Request Overview

This PR adds automatic spam detection and flagging functionality to the discussions feature. It introduces a new AutoSpamAlertBanner component that displays a warning when content is automatically flagged as potential spam, with an explanatory modal providing details about the automated flagging process.

Key changes:

Created AutoSpamAlertBanner component with danger alert banner and explanatory modal
Integrated spam detection banner into Post, Comment, and Reply components
Added internationalized messages for spam alert banner and modal content

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
src/discussions/common/AutoSpamAlertBanner.jsx	New component that renders spam warning banner with help modal
src/discussions/common/index.js	Exports the new AutoSpamAlertBanner component
src/discussions/posts/post/Post.jsx	Integrates spam banner, extracts is_spam from thread data
src/discussions/post-comments/comments/comment/Comment.jsx	Integrates spam banner, extracts is_spam from comment data
src/discussions/post-comments/comments/comment/Reply.jsx	Integrates spam banner with proper layout, extracts is_spam from comment data
src/discussions/messages.js	Adds internationalization messages for spam banner and modal

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/discussions/common/AutoSpamAlertBanner.jsx

src/discussions/messages.js

src/discussions/common/AutoSpamAlertBanner.jsx

src/discussions/posts/post/Post.jsx

src/discussions/post-comments/comments/comment/Reply.jsx

src/discussions/post-comments/comments/comment/Comment.jsx

Copilot

Pull Request Overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull Request Overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/discussions/common/AutoSpamAlertBanner.jsx

src/discussions/post-comments/comments/comment/Reply.jsx

Copilot

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/discussions/post-comments/comments/comment/Comment.jsx

src/discussions/common/AutoSpamAlertBanner.jsx

Copilot

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/discussions/post-comments/comments/comment/Comment.jsx

Copilot

Pull Request Overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/discussions/common/AutoSpamAlertBanner.jsx

Copilot

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/discussions/post-comments/comments/comment/Comment.jsx

e -i HEAD~4

chintanjoshi-apphelix-2u

A minor comment, is approved otherwise

Copilot

Pull Request Overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-11-11T05:56:02Z

src/discussions/post-comments/comments/comment/Comment.jsx

+  // If isSpam is not provided in the API response, default to false
+  const isSpamFlagged = isSpam || false;


[nitpick] The comment and the code logic are redundant. The expression isSpam || false already provides the default value when isSpam is undefined, null, or false. Either remove the comment as it describes obvious behavior, or simplify to const isSpamFlagged = Boolean(isSpam); if you want to be more explicit about type conversion.

Suggested change

// If isSpam is not provided in the API response, default to false

const isSpamFlagged = isSpam || false;

const isSpamFlagged = Boolean(isSpam);

Alam-2U and others added 8 commits October 6, 2025 19:30

Merge pull request openedx#808 from Alam-2U/ealam/LP-85

231c15a

fix: resolve InContext Sidebar post menu dropdown clipping that triggered scroll

feat: Add AutoSpamAlertBanner component

470f325

feat: Add auto spam messages and modal explanations

24eb0e3

feat: Add AutoSpamAlertBanner export to index.js

0afbb30

feat: Add AutoSpamAlertBanner to Comment component

6d5af8f

feat: Integrate AutoSpamAlertBanner for spam notifications

27b1c6c

Added AutoSpamAlertBanner component to display spam alerts.

feat: Integrate AutoSpamAlertBanner and update spam handling

d67623b

Added AutoSpamAlertBanner to Post component and adjusted spam flag logic.

Merge branch 'master' into COSMO2-771-ai-moderation-ui

4da3002

Copilot AI review requested due to automatic review settings November 7, 2025 12:26

Copilot AI reviewed Nov 7, 2025

View reviewed changes

santhosh-apphelix-2u added 2 commits November 7, 2025 18:15

feat: Fix isSpam flag default value in Comment component

5d2f979

feat: Fix variable naming and default value for isSpam

077fa3d

Copilot AI review requested due to automatic review settings November 7, 2025 12:46

Copilot AI reviewed Nov 7, 2025

View reviewed changes

Copilot AI review requested due to automatic review settings November 10, 2025 10:14

Copilot AI reviewed Nov 10, 2025

View reviewed changes

src/discussions/common/AutoSpamAlertBanner.jsx Show resolved Hide resolved

src/discussions/post-comments/comments/comment/Reply.jsx Outdated Show resolved Hide resolved

santhosh-apphelix-2u added 4 commits November 10, 2025 11:07

chore: Rename is_spam to isSpam and update default value

e1cebaf

chore: test

14735cc

chore: Linting error fix

4645a4d

chore: Linting error fix

f25cbf5

santhosh-apphelix-2u force-pushed the COSMO2-771-ai-moderation-ui branch from 1e079de to f25cbf5 Compare November 10, 2025 11:08

chore: Linting error fix

5a73e33

Copilot AI review requested due to automatic review settings November 10, 2025 11:12

Copilot AI reviewed Nov 10, 2025

View reviewed changes

src/discussions/post-comments/comments/comment/Comment.jsx Show resolved Hide resolved

src/discussions/common/AutoSpamAlertBanner.jsx Outdated Show resolved Hide resolved

santhosh-apphelix-2u added 2 commits November 10, 2025 11:20

chore: Linting error fix

f1a2143

chore: Linting error fix

a2c0887

Copilot AI review requested due to automatic review settings November 10, 2025 11:54

Copilot AI reviewed Nov 10, 2025

View reviewed changes

src/discussions/post-comments/comments/comment/Comment.jsx Outdated Show resolved Hide resolved

chore: Linting error fix

4bb0c8e

Copilot AI review requested due to automatic review settings November 11, 2025 02:55

Copilot started reviewing on behalf of santhosh-apphelix-2u November 11, 2025 02:55 View session

Copilot finished reviewing on behalf of santhosh-apphelix-2u November 11, 2025 02:58

Copilot AI reviewed Nov 11, 2025

View reviewed changes

src/discussions/common/AutoSpamAlertBanner.jsx Outdated Show resolved Hide resolved

chore: Linting error fix

12a6048

santhosh-apphelix-2u force-pushed the COSMO2-771-ai-moderation-ui branch from 782ccb8 to 12a6048 Compare November 11, 2025 03:02

chore: Linting error fix

72f432c

Copilot AI review requested due to automatic review settings November 11, 2025 03:19

Copilot started reviewing on behalf of santhosh-apphelix-2u November 11, 2025 03:19 View session

Copilot finished reviewing on behalf of santhosh-apphelix-2u November 11, 2025 03:20

Copilot AI reviewed Nov 11, 2025

View reviewed changes

src/discussions/post-comments/comments/comment/Comment.jsx Outdated Show resolved Hide resolved

chore: Linting error fix

0a2d471

chintanjoshi-apphelix-2u reviewed Nov 11, 2025

View reviewed changes

e -i HEAD~4 Outdated Show resolved Hide resolved

chintanjoshi-apphelix-2u requested changes Nov 11, 2025

View reviewed changes

chore: Delete e -i HEAD~4

c872894

Copilot AI review requested due to automatic review settings November 11, 2025 05:53

Copilot started reviewing on behalf of santhosh-apphelix-2u November 11, 2025 05:54 View session

Copilot finished reviewing on behalf of santhosh-apphelix-2u November 11, 2025 05:55

Copilot AI reviewed Nov 11, 2025

View reviewed changes

santhosh-apphelix-2u merged commit c79a583 into master Nov 13, 2025
9 checks passed

		// If isSpam is not provided in the API response, default to false
		const isSpamFlagged = isSpam \|\| false;

	// If isSpam is not provided in the API response, default to false
	const isSpamFlagged = isSpam \|\| false;
	const isSpamFlagged = Boolean(isSpam);

feat: Implement automatic spam detection banners #8

feat: Implement automatic spam detection banners #8

Conversation

santhosh-apphelix-2u commented Nov 7, 2025

Key Updates

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

chintanjoshi-apphelix-2u left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants