Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ability to close playlists within time period after creation #30401

Closed
peppy opened this issue Oct 23, 2024 Discussed in #30393 · 2 comments · Fixed by #30793
Closed

Ability to close playlists within time period after creation #30401

peppy opened this issue Oct 23, 2024 Discussed in #30393 · 2 comments · Fixed by #30793
Assignees
@bdach
Labels
area:playlists priority:1 Very important. Feels bad without fix. Affects the majority of users.

Comments

@peppy
Copy link
Member

peppy commented Oct 23, 2024

It's quite easy to make a mistake and want to recreate a playlist. I've had several requests for manual deletion for cases like this.

Discussed in #30393

Originally posted by Hiviexd October 23, 2024
would be a very appreciated UX improvement imo.

in multiple past cases related to spotlights/loved (most recently in osu!dev), we had to request manual edits/deletions to playlists due to mistakes during the creation process (wrong mods, title typo, wrong deadline). so it would be great if users were allowed to close their playlists within 5 minutes after creation, that'd help with solving any potential mistakes while preserving the long-term integrity of playlist leaderboards.
@peppy peppy added area:playlists priority:1 Very important. Feels bad without fix. Affects the majority of users. labels Oct 23, 2024
@ppy ppy deleted a comment Oct 28, 2024
@bdach bdach self-assigned this Nov 7, 2024
@bdach
Copy link
Collaborator

bdach commented Nov 7, 2024

How did you perform the 'deletion' logistically? Did you just set ends_at on the room, or did you actually delete rows? I was probably gonna do the former.

@peppy
Copy link
Member Author

peppy commented Nov 14, 2024

Just setting ends_at is fine I think. We can "actually" delete it if there's reason provided by users to do so.

This was referenced Nov 19, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bdach bdach

Labels
area:playlists priority:1 Very important. Feels bad without fix. Affects the majority of users.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add ability to close playlists within grace period after creation bdach/osu
2 participants
@peppy @bdach