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

Support maps #13

Open
GhoulofGSG9 opened this issue Oct 12, 2016 · 2 comments
Open

Support maps #13

GhoulofGSG9 opened this issue Oct 12, 2016 · 2 comments

Comments

@GhoulofGSG9
Copy link

GhoulofGSG9 commented Oct 12, 2016

Steamworks modifications that are maps should be supported as well. Allow the server ops to specify the map name via the config and add maps correctly to the map cycle (so the modid is stored with the map entry and not in the global mod field).

@keatsandyeats
Copy link
Owner

I haven't yet decided if I want to allow admins to add arbitrary workshop mods. If I disallow them from doing so, then I don't see the benefit of letting them add maps; the server op would be required to add the map to the whitelist, and if action is required by the op then he might as well just put the map in the mapcycle directly (and remove it from the rotation if he wishes).

If I do allow admins to add arbitrary maps, it probably won't be as good as a server op doing it. It might not be voteable, depending on how Shine MapVote is configured; it might not be in a good position on the list, if MapCycle is in order mode; and it might not have the right extra options that MapVote can look for. I may be able to let admins configure all this, but again I'm not sure whether admins should have this control.

I could add config options and let each op decide their admins' power. But adding map support properly might be more work than I am willing to do.

Thanks for your suggestion. I'm open to discussion on how much power to give admins.

@keatsandyeats keatsandyeats changed the title [Feature Request] Support maps Support maps Oct 12, 2016
@GhoulofGSG9
Copy link
Author

GhoulofGSG9 commented Oct 13, 2016

I requested this feature mostly because it's the use case that isn't covered by the web admin atm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants