-
Notifications
You must be signed in to change notification settings - Fork 454
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
Teacher lounge expansion WIP #2530
base: master
Are you sure you want to change the base?
Conversation
Holy crap, this is incredible! I'll check this out when I have time later, but so far it looks like an impressive amount of work. I know I keep saying I'm gonna jump back into hawkthorne, but I'm way too swamped with schoolwork right now and don't know when I'll be able to come back. Good news is, I'm properly learning how to do some legit programming so hopefully when I get back (and I will) I won't be submitting some amateur spaghetti code. |
ac24648
to
e91beab
Compare
Thanks @didory123 ! The more tester on this the better, I'd be interested to hear your thoughts. Looking forward to seeing you back around here. |
This is so awesome, So much new content! Couple of small issues I had,
I'm also thinking it be helpful for the player if we drop some hints about where to go next, for example after borchert gives the player some money it may be helpful if borchert mentions where the bursor can be found. |
733c187
to
1c32f4c
Compare
@CalebJohn everything you mentioned should be fixed. |
Great! I'll take a look at it tonight |
@@ -105,5 +100,16 @@ | |||
<property name="sprite" value="wastebin"/> | |||
</properties> | |||
</object> | |||
<object type="info" x="244" y="195" width="66" height="42"> | |||
<properties> | |||
<property name="info" value="Do to a mix up in the records room all student records have been lost. As you know, our student records are stored on a Microsoft Paint file, which the Dean was assured was future-proof. | Meanwhile, our extended warranty is not being honored by the restaurant that used to be Circuit City and we are asking every student to re-register for their classes."/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should say "Due to a mix up".
Besides the typo I made a comment on this all looks great! Great work @8bitgentleman lets get this merged in asap! |
6361aad
to
9e30407
Compare
Typo has been fixed. Since this is such a large addition I'd really like to get as many eyes on this as possible. If @didory123 @edisonout @niamu etc (sorry niamu I know you never want to touch lua again haha). could take a quick look and run-through of this hopefully we can minimize bugs/future bugfixes. |
Haven't had a proper run through yet, but from a quick play through it looks awesome. I've already run into a few bugs (spelling, npcs running away while they're talking, getting stuck in an endless loop of Shirley's sandwiches dialogue and not being able to get away). There's so much of season five & six I don't remember/never actually watched so I'm sure some of the weird stuff I don't get is a reference to something. Are we ever going to get round to fixing the quest module so we can have multiple quests/access quests from the command line? Because if so, I think we need to stop adding quests until that's done. I have some work stuff that I've been putting off that needs doing by Monday, so I'll probably procrastinate this weekend and do some in-depth testing instead. I know I haven't been around for a while to help out or test things but in future I think we should probably do little changes and tests at a time rather than a huge new number of levels/quests because though it's an awesome new feature, it's likely to have lots of bugs we miss because there's so much going on. |
Yea this pull kind of got away from me. I can definitely separate it into 3 pulls (1 for the lounge, 1 for the speakeasy and 1 for the bursar) if that would make things easier. If I remember correctly quests from the command line aren't possible/ hideously difficult to implement. I think it was @CalebJohn who I discussed this with. Looking forward to seeing all the ways you can break this. |
@8bitgentleman - don't worry about separating it out. We just need to think about keeping things small and separate in future. I think I've tested everything for the teachers lounge section (except spelling, grammar and sound). I have just tried playing it rather than trying to break it. Greendale exterior
Teachers lounge
Teahers lounge bathroom
Wings 1, 2 & 3
Computer Lab
|
Alright I've addressed some of the bug and concerns that you brought up:
Greendale exterior
Teachers lounge
Teachers lounge bathroom
Wings, 1, 2, 3
Lab
Thanks for playing through all this, I'd be interested to see what bugs you find in the other new areas. |
Thanks Gent. No worries about moving the quad. It was just an idea without much thought behind it. We should come up with something worth hiding in that third hedge hole though. I think NPCs facing the wrong direction is in the NPC code rather than anything you've done. I remember the alien quest guy being a pain with walking off while he was talking. Regarding the rope and flashlight, I wasn't sure if that had anything to do with the episode or was just something extra for the players to do before they got into the computer wing. I really should watch the later episodes again (I've watched season 5 once and still haven't watched all of 6 - no clue what that giant hand statue is about). I understand the rooms being empty but it just seems a drag for the player to go lounge->exterior->lounge->bathroom->lounge->wing-1->wing-2->wing-3 and they still haven't accomplished anything or get rewarded until they get into the computer lab. Maybe I'm just not used to extended quests. It's not really a criticism, just something I'm not familiar with. I hate making 2.5D rooms so I don't blame you. I just got confused when I tried to walk around and then ended up crouching instead. |
@8bitgentleman Hey Gent, what else is left to do for this pull? If I could help out in any way, let me know |
Hey @didory123 I updated the top with open issues. Also take a look at @edisonout and my comments. There are some levels that could use secrets/rewards for getting to them so figuring those out would be helpful too. |
Floorspace offsets and poster text still needed. Also some weirdness with the player moving through sprites
Key for the hidden door still doesn’t work Open the borchert pool when the quest is complete
Curtain fix
Speakeasy, hologram and improvement start Add the speakeasy, hologram, door fix, giant hand, fix for moving platforms with multi sprites and start improvements Bursar shopping state Add todd and goat
8f409e1
to
d121041
Compare
This adds a whole new set of areas to the Greendale campus including several improvements for the campus that can be purchased with Borcher’s money from the greendale bursar, the teacher's lounge, borchert pool and Shirley's Sandwiches. Right now there are still several things that need to be finished/resolved before this is ready.