Open
Conversation
amanuense
commented
Feb 13, 2018
| support_armory_available = false; | ||
| }; | ||
|
|
||
| if (isNil "spawn_beacon_global_available") then { |
amanuense
commented
Feb 13, 2018
| //add earplugs | ||
| player execVM "misc\simpleEP.sqf"; | ||
|
|
||
| 0 = 0 spawn { //make sure we add the handler post init. |
amanuense
commented
Feb 13, 2018
Owner
Author
amanuense
left a comment
There was a problem hiding this comment.
added some marking so you know what is new code
| sleep 15; | ||
| // SITREP | ||
| ["sitrepinfo",["SITREP","You can also save the game by giving a SITREP"]] call bis_fnc_showNotification; | ||
|
|
| publicVariable "support_armory_available"; | ||
| ["armory",["Armory Unlocked","Access the armory at the HQ and at the dropped supply crates"]] call bis_fnc_showNotification; | ||
| }; | ||
| if (_text == "Spawn beacon all players") then { |
| }; | ||
| }; | ||
|
|
||
| class spawn_beacon |
| @@ -0,0 +1,25 @@ | |||
|
|
|||
| removeAfterExpressionCall = 1; // 1 to remove the item after calling | ||
| }; | ||
| }; | ||
| class spawn_beacon |
Collaborator
|
@amanuense and I have had a look at this. There's a current issue when respawning due to the spawn height, as well as the time the spawn beacon exists which he's currently looking into. Apart from that, it's working! once the above are resolved this can be merged with master. @amanuense -will you be including the revive stuff we encountered yesterday in this? |
Owner
Author
|
Yes I'll include the fixes and changes
…On Mon, Jun 4, 2018, 14:09 Caius Ajiz ***@***.***> wrote:
@amanuense <https://github.com/amanuense> and I have had a look at this.
There's a current issue when respawning due to the spawn height, as well as
the time the spawn beacon exists which he's currently looking into.
Apart from that, it's working! once the above are resolved this can be
merged with master.
@amanuense <https://github.com/amanuense> -will you be including the
revive stuff we encountered yesterday in this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#24 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AL6Avaj9im8g_GgDfPifLnGgYotJ1F8Uks5t5aH5gaJpZM4SDgTu>
.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
implementation of issue #18
CODE IS NOT FULLY TESTED THIS IS A PRELIMINARY CODE REVIEW
I added some comments to the places I changed to add the menu rest of stuff is minor improvements to the code.