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

Reported issue fixes #573

Merged
merged 3 commits into from
Jul 14, 2016
Merged

Conversation

NanakoAC
Copy link
Contributor

Fixed hatch overlay not being cut when a firedoor opens
Fixed firedoor hatches drawing ontop of office/blast shutters when both are on the same tile
Drones can now bump open windoors
added attack logs for meteor impacts
slightly increased volume of meteor shield impact sound
fixed meteor event end announcements

Fixed firedoor hatches drawing ontop of office/blast shutters when both are on the same tile
Drones can now bump open windoors
added attack logs for meteor impacts
slightly increased volume of meteor shield impact sound
fixed meteor event end announcements
@NanakoAC
Copy link
Contributor Author

NanakoAC commented Jul 13, 2016

Lack of changelog is deliberate, since these fixes are for unreleased things, just merge them in silently
Fixes:
#563
#565
#566

@@ -142,6 +143,9 @@
!istype(A,/obj/machinery/field_generator) && \
prob(detonation_chance))
explosion(loc, power, power + power_step, power + power_step * 2, power + power_step * 3, 0)
msg_admin_attack("Meteor exploded at coords <[x],[y],[z]>")
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change all of these <[x],[y],[z]> references to this: (<a href='?_src_=holder;adminplayerobservecoodjump=1;X=[x];Y=[y];Z=[z]'>JMP</a>)

@skull132 skull132 added Changes Required The PR requires changes before it can be approved and/or merged. and removed Review Required labels Jul 13, 2016
@skull132
Copy link
Contributor

Doesn't satisfy my report on #565 . Read the last reply.

Also, one required change noted in the line comments.

@skull132
Copy link
Contributor

Disregard my post about 565, it appears to be BYOND client related and works fine on 510 build. The coords need fixing, tho.

@@ -131,7 +131,7 @@

if (T)//We have a double safety check on T to prevent runtime errors
meteor_shield_impact_sound(T, shieldsoundrange)
msg_admin_attack("Meteor impacted energy field at coords <[x],[y],[z]>")
msg_admin_attack("Meteor impacted energy field at coords (<a href='?_src_=holder;adminplayerobservecoodjump=1;X=[x];Y=[y];Z=[z]'><[x],[y],[z]>JUMPTO</a>)")
Copy link
Contributor

Choose a reason for hiding this comment

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

I told you specifically to use JMP. Every other jump link ingame is formatted like that.

@NanakoAC NanakoAC mentioned this pull request Jul 14, 2016
@NanakoAC NanakoAC added Review Required and removed Changes Required The PR requires changes before it can be approved and/or merged. labels Jul 14, 2016
@skull132 skull132 merged commit de4e73b into Aurorastation:development Jul 14, 2016
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

2 participants