ppy / osu-web

the browser-facing portion of osu!
https://osu.ppy.sh
GNU Affero General Public License v3.0
982 stars 385 forks source link

Display temporary tournament bans on user profiles #11590

Closed hburn7 closed 1 day ago

hburn7 commented 3 weeks ago

This is directly related to #9018 and #9019. I'm raising a new issue as I feel the functionality of displaying temporary tournament bans was missed when implementing this feature, thus this is a bug report. If it is a deliberate decision to not display temporary tournament bans, I would be interested to know why, as both #9018 and #9019 display temporary bans in their UI mockups (and a vast majority of tournament bans are temporary).

Currently, users with temporary tournament bans do not have a matching entry in the account history section.

Albionthegreat commented 3 days ago

Having temporary tournament bans displayed on user profiles would be incredibly useful. I believe there are multiple benefits, making this more than worthwhile.

Having indefinite tournament bans displayed on player profiles is great, and I believe all parties would benefit from doing the same for temporary ones. This is something we absolutely want.

notbakaneko commented 3 days ago

Non permanent bans aren't displayed after a certain period of time (I believe the default is 28 days?). So is the issue that there are temporary bans that are longer than the 28 days and thus won't be displayed even if they're still active?

Hiviexd commented 3 days ago

solution could be to display non-permanent infractions for their entire duration, with a minimum of 28 days to maintain current behavior for most common cases.

peppy commented 2 days ago

Sounds fair.

nanaya commented 2 days ago

the table doesn't have explicit end time though, only start time and duration.

Hiviexd commented 2 days ago

can't end time be inferred from both?

nanaya commented 2 days ago

yeah, I guess with the permanent flag addition it already needs to check all rows anyway.

nanaya commented 2 days ago

that said, there's exactly one non-permanent (permanent = 0) tournament ban (ban_status = 3) entry in the database right now. And it expired already last year.

Are the others stored somewhere else?

Hiviexd commented 2 days ago

we were instructed to avoid applying non-permanent tournament bans because:

  1. they apply silences on users
  2. there's currently no way to remove them
  3. they stop displaying after 28 days even when still active

1 and 2 are most likely ASS-related, so for osu-web 3 (and possibly 1?) need to be addressed.

nanaya commented 2 days ago

The number 3 seems weird because moderators admins should still be able to see everything and setting it as anything else means only moderators admins can see it (and no easy way to fix it now?).

Userlog2 commented 2 days ago

Sounds good, because:

  1. It allows hosts to check e.g. emergency refs for tournament bans without having to rely on the player
  2. It allows non-officially supported (unbadged) hosts to consider tournament bans, including those stemming from recent restrictions

The issues Hivie points out should absolutely be fixed, though.