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.

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

Remove Beatmap_ID and Beatmapset_ID articles #4802

Merged
merged 1 commit into from
Jan 19, 2021
Merged

Remove Beatmap_ID and Beatmapset_ID articles #4802

merged 1 commit into from
Jan 19, 2021

Conversation

TicClick
Copy link
Contributor

part of #4723, they are replaced by /wiki/Beatmap#Identification now

@Joehuu
Copy link
Member

Joehuu commented Jan 19, 2021

I feel like this is ruining this idea said in #3143:

A wiki thrives and lives from having each article be intertwined with each other. It lives from the idea that you can go to an article you find or look up and you can follow down the rabbit hole of information by clicking every important term and be sent to a new article that tells you more about it.

Not sure if this applies to "beatmap(set} id" as it is not really a common word, but still.

@TicClick
Copy link
Contributor Author

I'd agree with you, but it's more of an internal infomation that only needs to be mentioned briefly (besides, these two are really similar with the exception of the terms themselves and 0/-1 being default values): #4711 (comment)

@Joehuu Joehuu merged commit 57ce7c8 into ppy:master Jan 19, 2021
@TicClick TicClick deleted the en-remove-beatmap-id branch January 19, 2021 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants