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

Update BN Pages + (also staff log, gmt, alumni) #2612

Merged
merged 7 commits into from
Sep 27, 2019
Merged

Update BN Pages + (also staff log, gmt, alumni) #2612

merged 7 commits into from
Sep 27, 2019

Conversation

NoffyNoffeh
Copy link
Contributor

@NoffyNoffeh NoffyNoffeh commented Sep 26, 2019

  • Update 2019 staff log
  • Update bn page
  • Move "becoming a bn" to its own dedicated page
  • Update "general information for bns" (more information will be in a later pr, but i feel the current page is too outdated to let just sit there)

edit:

  • Marcin alumni since that happened soon after opening pr and impacts staff log page

* Staff Log end of September update

* Create "becoming BN page"

* update BN page

update team section, move "becoming a bn" section

* old page for clayton

* update general info (a bit)
* marcin on staff log

* marcin gone from gmt

* marcin alumni
@NoffyNoffeh NoffyNoffeh changed the title Update BN Pages Update BN Pages + (also staff log, gmt, alumni) Sep 26, 2019
Copy link
Member

@cl8n cl8n left a comment

Choose a reason for hiding this comment

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

didn't read the bulk of the text yet, I'll do that soon

wiki/People/Beatmap_Nominators/en.md Outdated Show resolved Hide resolved
wiki/People/osu!_Alumni/en.md Outdated Show resolved Hide resolved
* A -> a

* move below props

* top links

* remove space
Copy link
Member

@cl8n cl8n left a comment

Choose a reason for hiding this comment

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

lazy suggestions

wiki/People/Beatmap_Nominators/General_Information/en.md Outdated Show resolved Hide resolved
wiki/People/Beatmap_Nominators/General_Information/en.md Outdated Show resolved Hide resolved
wiki/People/Beatmap_Nominators/General_Information/en.md Outdated Show resolved Hide resolved
wiki/People/Beatmap_Nominators/General_Information/en.md Outdated Show resolved Hide resolved
wiki/People/Beatmap_Nominators/General_Information/en.md Outdated Show resolved Hide resolved
@Pachiru
Copy link
Contributor

Pachiru commented Sep 27, 2019

I may be late, but I think that's quite useless to keep the flame, nuke, bubble pop, bubble, heart and such, since the ranking process doesn't really work like that anymore. (Unless you want to keep it as a history thingy)

@NoffyNoffeh
Copy link
Contributor Author

Yes the plan is keeping that information in an unused folder for now so we can easily place it in a more fitting history section after reworking the mapping pages and figuring out a good place for it to go.

* move history unused to its own folder

* update becoming bn page

* upload images for "icons"

* add "icons" and update general info page
@cl8n cl8n merged commit e9ed583 into ppy:master Sep 27, 2019
@cl8n
Copy link
Member

cl8n commented Sep 27, 2019

if any other reviewers were looking at this-- it was meant to be a quick update so noffy &pishi don't have to work around all of this old information for #2615 , that's why the language is still a bit strange as in the original articles

we will revisit this soon:tm:

@NoffyNoffeh NoffyNoffeh deleted the Update-BN-Pages branch September 28, 2019 17:11
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.

4 participants