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

ACRE Integration, require test and adapt #327

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

StefArma
Copy link
Member

@StefArma StefArma commented May 1, 2018

I don't have ACRE, if anyone could test it

Code imported from WotP, thanks Barbolani!

@StefArma StefArma added the WIP Work In Progress label May 1, 2018
@StefArma StefArma added this to the 1.8.1 milestone May 1, 2018
@Mcfluffles
Copy link

I've tested this today, and seems there is some issues. All I've done is download the ACRE Integration branch, packed the PBO, and uploaded it to our group's server to test it. Seems you have some script errors,
Server RPT Log: https://pastebin.com/mLPf9VbZ
screens here: https://steamcommunity.com/sharedfiles/filedetails/?id=1378035808
https://steamcommunity.com/sharedfiles/filedetails/?id=1378035797
This is the full mission folder I packed + PBO: https://drive.google.com/drive/folders/1vTBsObP5HJt2zRH2BAApcZ1qUYLhCPpA?usp=sharing

Hope it helps.

@StefArma
Copy link
Member Author

StefArma commented May 5, 2018

Hello! Thanks a lot! i'll make a fix

@Mcfluffles
Copy link

Unfortunately still doesn't seem to be working. Still does not detect having a radio, even if you have an ACRE radio in your inventory, so one cannot call in NATO support. Also, some script errors still exist:
https://pastebin.com/j01VGzEp

Sketchy Irishman also seems to be bugged, cannot buy from him:
http://steamcommunity.com/id/mcfluffles/screenshot/934938345592846726

@zalexki
Copy link
Contributor

zalexki commented May 9, 2018

Need rebase

@zalexki
Copy link
Contributor

zalexki commented May 11, 2018

Let's keep this for a few but #346 will probably replace this one.

@StefArma
Copy link
Member Author

yes that's way better

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

Successfully merging this pull request may close these issues.

3 participants