mirror of
https://github.com/minetest/contentdb.git
synced 2025-01-08 22:17:34 +01:00
Add Editor crash course (#385)
Co-authored-by: rubenwardy <rw@rubenwardy.com>
This commit is contained in:
parent
3be937c503
commit
c0447cdcd2
@ -15,8 +15,9 @@ Editors should make sure they are familiar with the
|
||||
|
||||
## ContentDB is not a curated platform
|
||||
|
||||
It's important to note that ContentDB isn't a curated platform, but it also does have some
|
||||
requirements on minimum usefulness. See 2.2 in the [Policy and Guidance](/policy_and_guidance/).
|
||||
It's important to note that ContentDB isn't a curated platform - a mod doesn't need to be good
|
||||
to be accepted, but there are some minimum requirements when it comes to usefulness and other things.
|
||||
See 2.2 in the [Policy and Guidance](/policy_and_guidance/).
|
||||
|
||||
## Editor Work Queue
|
||||
|
||||
@ -32,3 +33,26 @@ The [Editor Work Queue](/todo/) and related pages contain useful information for
|
||||
Editors currently receive notifications for any new thread opened on a package, so that they
|
||||
know when a user is asking for help. These notifications are shown separately in the notifications
|
||||
interface, and can be configured separately in Emails and Notifications.
|
||||
|
||||
## Crash Course to being an Editor
|
||||
|
||||
The [Package Inclusion Policy and Guidance](/policy_and_guidance/) is our go to resource for making decisions in
|
||||
changes needed, similar to how lua_api.txt is the doc for modders to consult.
|
||||
|
||||
In the [Editor console](/todo/), the two most important tabs are the Editor Work Queue and the Forum
|
||||
Topics tab. Primarily you will be focusing on the Editor Work Queue tab, where a list of packages to review is.
|
||||
|
||||
When you have some free time, feel free to scroll through the Forum Topics page and import mods into ContentDB.
|
||||
But don't import a mod if it's broken, outdated, not that useful, or not worth importing - click Discard instead.
|
||||
|
||||
A simplified process for reviewing a package is as follows:
|
||||
|
||||
1. scan the package image if present for any obvious closed source assets.
|
||||
2. if right to a name warning is present, check its validity and if the package meets
|
||||
the exceptions.
|
||||
3. if the forums topic missing warning is present, feel free to check it, but its
|
||||
usually incorrect.
|
||||
4. check source, etc links to make sure they work and are correct.
|
||||
5. verify that the package has license file that matches what is on the contentdb fields
|
||||
6. verify that all assets and code are licensed correctly
|
||||
7. if the above steps pass, approve the package, else request changes needed from the author
|
||||
|
Loading…
Reference in New Issue
Block a user