Terasology / GooeysQuests

Go on quests with Gooey, our official mascot!
Apache License 2.0
2 stars 18 forks source link

Added Dungeon maze #2

Closed VaibhavBajaj closed 7 years ago

VaibhavBajaj commented 7 years ago

Purpose

I have added dungeonMaze and two blocks: hardStoneIllusion block and mantleStoneIllusion block. I added mantle stone illusion block since I planned to make the maze with it since a maze that is destructible is not a maze at all. Mantle stone was much too dark and didnt blend in so I made it with hardStone instead.

Things to keep in mind while testing

I might point out that there are 5 chests in this building. 2 of these 5 are behind illusion blocks. Please test that you can recieve items from these chests. The one near the staircase is particularly hard to find since you can't even see the chest, simply the blocks. You must go into the illusion blocks near the staircase and press E. You will see the contents of the chest with what I believe is an ArchmageFryingPan (my addition from a few weeks ago :P).

Pictures of maze

The maze Inside the maze Top floor

Cervator commented 7 years ago

Merged - very well done :-)

Worked first try (well, not first world, had to go through a few before it spawned), few commits, good concept, clean code. You're getting good!

Only one oddity: leaving in the TODO in one of the JSON files after already fixing it (by moving one of the files). Really short term todos like that are better as comments on an issue or PR. TODOs in code are more for "Eh maybe one day somebody else will be working in here and feel like also doing x"