Skip to content

Conversation

haileyok
Copy link
Member

No description provided.

Copy link

github-actions bot commented Sep 25, 2025

Old size New size Diff
9.41 MB 9.48 MB 75.6 KB (0.78%)

@layeredstrange
Copy link

Line 276 of src/components/moderation/ReportDialog/utils/useReportOptions.ts seems to have a spelling mistake...

title: _(msgImpersation of Electoral Officials/Entities),

Just thought I'd let you know.

@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 04:47 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 04:49 — with Render Destroyed
Comment on lines +5 to +13
export type ReportCategory =
| 'childSafety'
| 'violencePhysicalHarm'
| 'sexualAdultContent'
| 'harassmentHate'
| 'misleading'
| 'ruleBreaking'
| 'civicIntegrity'
| 'other'
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are no longer types in Ozone, so we'll define them here.

Copy link
Contributor

@surfdude29 surfdude29 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed a few typos, style nits and places where the strings didn't match what was in the Moderation Report Granularity proposal. And I also I made a few suggestions where I think the language could be clearer and more helpful to users.

@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:16 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:16 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:16 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:17 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:17 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:17 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:17 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:17 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:17 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 20:17 — with Render Destroyed
@haileyok
Copy link
Member Author

@surfdude29 thanks! neovim doesn't have spellcheck :((

@surfdude29
Copy link
Contributor

@haileyok No worries :)

Not sure if you saw the other suggestions btw, some get hidden by GH 👀

@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 21:43 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 21:44 — with Render Destroyed
@arcalinea arcalinea temporarily deployed to hailey/new-report-reasons - social-app PR #9079 September 25, 2025 21:44 — with Render Destroyed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants