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

Minor Club remap: added windoor in front of booze dispenser, replaced… #5450

Closed
wants to merge 5 commits into from
Closed

Minor Club remap: added windoor in front of booze dispenser, replaced… #5450

wants to merge 5 commits into from

Conversation

maikilangiolo
Copy link
Contributor

About The Pull Request

Minor Club remap: added windoor in front of booze dispenser, replaced three walls with windows, added 'take-out' table to kitchen.

Condi-master and sink have been switched, since the sink looked ugly on the window and this way it is also closer to the cooking area.

(p.s. I got no clue how to make pull requests and operate git, so this PR might be fucked.)

Why It's Good For The Game

Well, it allows people to see inside the kitchen, and even come pick up food if they order before hand. Not only does it allow customers to see the chef cooking (and the chef can see customers that are waiting for him to come out), and adds the possibility to chat through the windows, but it also (hopefully) should reduce the instances of theft because people don't see anyone manning the club counter and climb it to steal drinks or food by breaking into the kitchen.

It does not make breaking into club any easier, since it would be faster to bolt open the door or smash it with a crowbar, or even slide under the flaps, rather than break the new windows/windoors and clib over the table or low wall.

Changelog

🆑

tweak: changed three walls in the kitchen to windows
tweak: switched condi-master and sink positions
tweak: moved the air alarm to above the drink showcase
add: added a new double windoor set-up for take-out
add: added windoor in front of booze dispenser to reduce or delay theft

/:cl:

… three walls with windows, added 'take-out' table to kitchen.
@maikilangiolo
Copy link
Contributor Author

forgot to link the image in the PR: https://ibb.co/9380DRP

@fernandos13
Copy link
Contributor

so one problem in the file. cev eris map. theres some lines with step_x = -2 you can just text eddit with VS or even notepad++ and remove it. literally delete this lines"

just the step_x= number

part of it. and any , or ; that follows. it will solve the compiling error

@@ -25129,7 +25129,8 @@
/area/eris/maintenance/section2deck4central)
"beo" = (
/obj/machinery/light{
dir = 1
dir = 1;
step_x = -2
Copy link
Contributor

Choose a reason for hiding this comment

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

like here. just delete the line with step_x = -2

Copy link
Contributor

Choose a reason for hiding this comment

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

and every other line thats like this one

Copy link
Contributor

Choose a reason for hiding this comment

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

solved

@fernandos13
Copy link
Contributor

i made the changes in a file
heres the file clean of ### step_x= something and step_x= something
_CEV_Eris.zip

@fernandos13
Copy link
Contributor

i made a PR to your branch. with its changes

@Firefox13
Copy link
Contributor

(merge conflict)
What I do suggest, is to add shutters to the windows that start closed. And a button in kitchen to control them.

Copy link
Contributor

@genessee596 genessee596 left a comment

Choose a reason for hiding this comment

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

Delegate stamp of approval.

The takeout window makes it easier for people to exchange food at the kitchen counter without having to wade through the numerous people (read: almost none) that are always sitting at the bar. A good QoL change.

Needs maintainer review before being mergable.

Copy link
Contributor

@Iglpigl Iglpigl left a comment

Choose a reason for hiding this comment

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

Resolve conflicts, looks good otherwise.

@maikilangiolo
Copy link
Contributor Author

fernando should have solved my own clumsiness and the conflicts that spawned when I mapped? I honeslty got no clue what's wrong, I just mapped some shit in and this happened.

@Firefox13
Copy link
Contributor

fernando should have solved my own clumsiness and the conflicts that spawned when I mapped? I honeslty got no clue what's wrong, I just mapped some shit in and this happened.

Mergeconflict happens when someone's PR edits the file your PR edits and it gets merged before yours.

@maikilangiolo
Copy link
Contributor Author

so, can I solve it or I can happily leave the task to someone that knows what he's fucking doing?

@Firefox13
Copy link
Contributor

so, can I solve it or I can happily leave the task to someone that knows what he's fucking doing?

yes, you need to save your changes(outside of the repo), rebase(update to master, it will wipe your chanhes) and re-apply them again

@maikilangiolo
Copy link
Contributor Author

so, how do I save the changes then? copy pasting the map file somewhere else will just use the old map before the rebase when I overwrite it.

@Firefox13
Copy link
Contributor

so, how do I save the changes then? copy pasting the map file somewhere else will just use the old map before the rebase when I overwrite it.

You copypaste the map outside of the repo
you update
you shove it back, but do not replace the map file - you manually ctrl-c ctrl-v changes into actual repo map that was updated

@maikilangiolo
Copy link
Contributor Author

i fucked up my git everything, fork included. so I will just close this PR, nuke my fork, eat a bullet and start again.

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

Successfully merging this pull request may close these issues.

None yet

6 participants