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

Reintroduce map phase XML tag #891

Merged
merged 3 commits into from
Jun 20, 2021
Merged

Reintroduce map phase XML tag #891

merged 3 commits into from
Jun 20, 2021

Conversation

applenick
Copy link
Member

@applenick applenick commented Jun 19, 2021

Reintroduce map phase XML tag

This PR reintroduces the map phase tag, allowing for map authors to identify whether their map is in a DEVELOPMENT or PRODUCTION phase. In addition to including the xml tag, the phase will now be displayed under /map for those with the debug perm. Also /maps can now be filtered by using the -p [phase] flag. Maps in the development phase are hidden from /maps by default.

XML Example

<phase>development</phase> # Identify the map is in a developmental stage
<phase>production</phase> # Identify the map is in a production stage (production is the default value)

/maps and /map Example

Example of /map when a map is in the PRODUCTION phase
production

Example of using /map which now has a -p [phase] flag
maps

As always, please let me know if there's any suggestions! 👍

Signed-off-by: applenick applenick@users.noreply.github.com

Signed-off-by: applenick <applenick@users.noreply.github.com>
@applenick applenick added the reintroduce Reintroduces a feature that was removed label Jun 19, 2021
@applenick applenick mentioned this pull request Jun 19, 2021
Signed-off-by: applenick <applenick@users.noreply.github.com>
Signed-off-by: applenick <applenick@users.noreply.github.com>
@Electroid Electroid merged commit 0a0ca91 into PGMDev:dev Jun 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reintroduce Reintroduces a feature that was removed
Development

Successfully merging this pull request may close these issues.

3 participants