Skip to content

Add "Hunting season" by solars, Peters and -sniper-#24

Closed
JostP wants to merge 9 commits into
MT-CTF:masterfrom
JostP:hunting_season
Closed

Add "Hunting season" by solars, Peters and -sniper-#24
JostP wants to merge 9 commits into
MT-CTF:masterfrom
JostP:hunting_season

Conversation

@JostP
Copy link
Copy Markdown
Member

@JostP JostP commented Feb 11, 2019

Hello
I made new map. I made it from part of solars' Karsthafen map. Its name is hunting season. It is small village made of wood and red stone.

Screenshots:

zaslonska slika 2019-02-11 17-40-47
zaslonska slika 2019-02-11 17-41-33
zaslonska slika 2019-02-11 17-41-02
zaslonska slika 2019-02-11 17-40-23

@Thomas--S
Copy link
Copy Markdown
Member

This village is called "Jägerhof" in the original map, isn't it?

Comment thread hunting_season.conf Outdated
@@ -0,0 +1,13 @@
name = Hunting season
author = solars, Peters and -sniper-
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Actually, solars asked for this line: (https://forum.minetest.net/viewtopic.php?f=12&t=6642&start=275#p341671)

Suggested change
author = solars, Peters and -sniper-
author = solars, Peters and -sniper- from the minetest.net forum

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Feb 12, 2019

What do you think about changing author and adding hint like this:
author = -sniper-
hint = Map is adapted from a Minetest world made by solars and Peters. You can find it on forum.minetest.net. Map is distributed under the CC-BY-SA 4.0 license

Tell me what do you think. Is hint correct in english?

map

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Feb 12, 2019

This village is called "Jägerhof" in the original map, isn't it?

What does "Jägerhof" mean? I didn't found a translation. I think it is better to give map english name.

@Thomas--S
Copy link
Copy Markdown
Member

What do you think about changing author and adding hint like this:
author = -sniper-
hint = Map is adapted from a Minetest world made by solars and Peters. You can find it on forum.minetest.net. Map is distributed under the CC-BY-SA 4.0 license

Tell me what do you think.

I think this is a good idea, but ultimately it's solars's decision.


Jägerhof means something like Hunters' Yard. Of course, I prefer the English name provided by you (Hunting Season) over the original name.


I already reviewed this map and I think it looks very promising.
As soon as the .conf file is fixed according to solars's wishes, I'll approve this map.

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Feb 12, 2019

As soon as the .conf file is fixed according to solars's wishesAs soon as the .conf file is fixed according to solars's wishes

https://creativecommons.org/licenses/by-sa/4.0/deed.en :
Attribution — You must give appropriate credit, provide a link to the license, and indicate if changes were made. You may do so in any reasonable manner, but not in any way that suggests the licensor endorses you or your use.

I am author of CTF map and solars and Peters are authors of the original map.
I can ask solars if you want.

@Thomas--S
Copy link
Copy Markdown
Member

I think it's best to ask solars.

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Feb 12, 2019

update Add hint and change "author" key

Do you have any suggestion about changing hint? Please check if hint is written correct in english.

Thomas--S
Thomas--S previously approved these changes Feb 12, 2019
@ClobberXD
Copy link
Copy Markdown
Contributor

I'm not happy with such a large hint. A hint is meant to provide players with a tip on how the map is meant to be played. The author field is logically more suitable for this, but even that isn't the best choice - keep in mind that both the author field and the hint field will be displayed in game. I prefer to not have large fields as these flood the chat at the start of every match. It also looks like the attribution should link to the license itself, which makes it even more complicated.

At the moment I can think of having a /maps chat-command that displays a formspec that lists all maps, with name, author, license, etc. We could then add a description field to the conf file that will be displayed in this formspec.

I could work on this, but the map can't be merged until then.

Copy link
Copy Markdown
Contributor

@ClobberXD ClobberXD left a comment

Choose a reason for hiding this comment

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

Please prefix the file names by 12_.

Nvm, I'll do that manually after merging. I wish we got rid of this naming scheme...

@ClobberXD ClobberXD dismissed stale reviews from Thomas--S and themself February 12, 2019 16:28

Invalid

@ClobberXD
Copy link
Copy Markdown
Contributor

Accidentally dismissed approval of @Thomas--S. Apologies.

@ghost
Copy link
Copy Markdown

ghost commented Feb 12, 2019

Why does numbering matter btw? Does it decide the sequence?

@ClobberXD
Copy link
Copy Markdown
Contributor

Why does numbering matter btw? Does it decide the sequence?

Yes, see map parsing function here - prefixing by numbers ensures that the maps are registered in the same order.

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Feb 12, 2019

Why does numbering matter btw? Does it decide the sequence?

Yes, see map parsing function here - prefixing by numbers ensures that the maps are registered in the same order.

So if I understand, forest map will be before plain battle and river valley before forest map ?

@ClobberXD
Copy link
Copy Markdown
Contributor

So if I understand, forest map will be before plain battle and river valley before forest map ?

Not really, the order of registration is the order in which the maps were merged, not the PRs created. So ideally, the number to be prefixed can only be decided when merging.

AFAIK, maps would still work without prefixing the filenames with numbers, but they just won't be consistent with respect to order of registration.

@BrianGaucher
Copy link
Copy Markdown

At the moment I can think of having a /maps chat-command that displays a formspec that lists all maps, with name, author, license, etc. We could then add a description field to the conf file that will be displayed in this formspec.

I agree with this. It would allow for longer descriptions, avoid licensing issues, provide useful hints, and would be good to let players see all the maps.

@ClobberXD
Copy link
Copy Markdown
Contributor

I'm not quite sure how the modifications field would help. We could keep it until the /maps feature has been merged, but then players won't be able to see/access the license from within the game, and that'd be a violation of the license.

This map, and the other ones from Karsthafen will have to be put on hold.

Comment thread hunting_season.conf Outdated
Co-Authored-By: Thomas--S <Thomas--S@users.noreply.github.com>
@Thomas--S
Copy link
Copy Markdown
Member

94e7fbf

@Thomas--S Thomas--S closed this Jun 22, 2019
@JostP JostP deleted the hunting_season branch July 6, 2019 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New map New map submission Two approvals 🎉 Waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants