Skip to content

Conversation

AlexTMjugador
Copy link
Member

Overview

This is a follow-up to PR #4426. I initially didn't consider the organizations an user belongs to as worth hiding, but given that user profiles can be public, I suppose there technically is a way to exploit them for SEO abuse. Overall, it also seems more consistent to hide them here too.

This is a follow-up to PR #4426. I initially didn't consider the
organizations an user belongs to as worth hiding, but given that user
profiles can be public, I suppose there technically is a way to exploit
them for SEO abuse. Overall, it also seems more consistent to hide them
here too.
@AlexTMjugador AlexTMjugador requested review from triphora and a team September 30, 2025 11:23
@AlexTMjugador AlexTMjugador added the backend Relates to Modrinth Backend or API label Sep 30, 2025
Copy link
Contributor

@aecsocket aecsocket left a comment

Choose a reason for hiding this comment

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

Could have been split into multiple PRs for easier review but approved

@AlexTMjugador AlexTMjugador added this pull request to the merge queue Sep 30, 2025
Merged via the queue into main with commit 895b040 Sep 30, 2025
3 checks passed
@AlexTMjugador AlexTMjugador deleted the alex/hide-hidden-orgs-from-user-profiles branch October 1, 2025 08:58
fetchfern pushed a commit that referenced this pull request Oct 1, 2025
This is a follow-up to PR #4426. I initially didn't consider the
organizations an user belongs to as worth hiding, but given that user
profiles can be public, I suppose there technically is a way to exploit
them for SEO abuse. Overall, it also seems more consistent to hide them
here too.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Relates to Modrinth Backend or API
Development

Successfully merging this pull request may close these issues.

2 participants