-
Notifications
You must be signed in to change notification settings - Fork 81
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
Rename "missing" pages into "curation" pages #1126
Comments
@Daniel-Mietchen, is this a proposal, or already approved and the patch just needs writing? |
At this point, it is just a proposal — we have not discussed this yet. I hope to get some feedback on this today. |
I like the idea. |
My motivation here is that "/missing" is a bit too ambiguous. When demoing the site to new users, phrases like "the missing pages for organizations" are then potentially misleading. Calling them "curation" would make their purpose clearer. |
This has several components that need to be coordinated:
|
As just discussed, we are going to treat this ticket as being about 1-3, opening a new one for 4, which is #1322. |
As just discussed, I will work on 1 and 2, and once that is set up, @egonw will do 3. |
@egonw I think I am through with the renaming — see https://github.com/fnielsen/scholia/tree/origin/feat/curation-pages-1126 for current state. |
Just for the record, we now have a pull request for this at #1324 . |
Sorry to nitpick, but is it on purpose that the URL is only available without the final /? If so, maybe a redirect should be set For example https://scholia.toolforge.org/topic/Q202864/curation vs https://scholia.toolforge.org/topic/Q202864/curation/ |
It seems that this is the behaviour on most pages (like any with a Q number). This seems to be a bigger than this so have created #1498 |
The text was updated successfully, but these errors were encountered: