Forged Alliance Forever

Re: Coming in patch 3635

2014-10-23T01:30:51+02:00

Ze_PilOt wrote:
The git is used for development, but as no team is made yet, the changes commited there are not approved by anyone but the owner of the repository (who is not the owner of the server).

The only named owner of the Github repository is visionik, who is (I believe) the owner of the server...

Statistics: Posted by IceDreamer — 23 Oct 2014, 01:30

Re: Coming in patch 3635

2014-10-22T20:45:49+02:00

Not sure if this is inside the scope of this patch, but I think it would be a great improvement if mass extractors had a higher target priority than mass storage.

I'm not sure if this is really "balance" change, just a convenience one, as whenever you do a runby you have to manually target the mex's and a bunch of the damage gets wasted.

Oh, and aeon t1 bombers will still target amphibious units, such as the ACU or wagers, while they are underwater.

Statistics: Posted by Nombringer — 22 Oct 2014, 20:45

Re: Coming in patch 3635

2014-10-22T20:36:28+02:00

The official and controlled repository is the bitbucket one.

The git is used for development, but as no team is made yet, the changes commited there are not approved by anyone but the owner of the repository (who is not the owner of the server).

Statistics: Posted by Ze_PilOt — 22 Oct 2014, 20:36

Re: Coming in patch 3635

2014-10-22T20:32:29+02:00

Github is where all primary development is being driven.

Statistics: Posted by IceDreamer — 22 Oct 2014, 20:32

Re: Coming in patch 3635

2014-10-22T15:41:23+02:00

Just so we are clear, which repository are we using now:

Visionick's Bitbucket
or
GitHub

I see Github has some recent changes, but Visionick's Bitbucket is listed in the sticky.

Statistics: Posted by The Mak — 22 Oct 2014, 15:41

Re: Coming in patch 3635

2014-10-22T10:40:05+02:00

Patch 3635 was released, then withdrawn shortly after, and now 3636 is a bit different so it has new number.

Statistics: Posted by pip — 22 Oct 2014, 10:40

Re: Coming in patch 3635

2014-10-22T09:10:01+02:00

What happened to patch 3635 ? Next patch seems to be numbered 3636.

Statistics: Posted by Krapougnak — 22 Oct 2014, 09:10

Re: Coming in patch 3635

2014-10-14T07:54:11+02:00

Shadows wrote:
Sheeo wrote:
Sir Prize wrote:Thank you devs!

I PMed Sheeo about a month ago I think asking if T1 Mass Extractors could have their strategic icon separated in the blueprints from T1 Mass Storage. It sounded like it was easy to do and would get up - any update on this?

Thanks again for your work guys.

Of course, I'll make sure to push this change later today

EDIT: It's done here: https://bitbucket.org/thepilot/forged-alliance-forever-lua/pull-request/116/new-icon-name-for-mass-storage

The link is broken.

The FAF repository is now in visionik hands:

https://bitbucket.org/visionik/forged-a ... ss-storage

Statistics: Posted by Ze_PilOt — 14 Oct 2014, 07:54

Re: Coming in patch 3635

2014-10-13T20:43:42+02:00

Sheeo wrote:
The change that was previously implemented (It's not on github), duplicated the icon along with its name. So we would have this issue with people using old mods.

The change is only to allow ASI to do this, so if you have an idea of what to do please let me know

In order to keep compatibility with older icon mods, the implementation of my idea should look like this:

Code:: UnitBlueprint { Merge = true, BlueprintId = "ueb1106", StrategicIconName = get_name_if_file_exist('icon_structure1_mass_storage', StrategicIconName) } function get_name_if_file_exist(new_file_name, default_file_name) local file_found=io.open(new_file_name, "r") if file_found==nil then io.close(file_found) return default_file_name else io.close(file_found) return new_file_name end end

The function sets the new icon's file name if the icon file exist, otherwise it leaves it at default.
(LUA is not my programing language).

Statistics: Posted by Shadows — 13 Oct 2014, 20:43

Re: Coming in patch 3635

2014-10-13T18:58:08+02:00

zeroAPM wrote:
A question, when and where were the changes discussed?

In different topics all over the forums at different times. Quite difficult to track down.

Previously we had discussion forums for new patches, but there wasn't one for 3635/3636.

Statistics: Posted by Sheeo — 13 Oct 2014, 18:58

Re: Coming in patch 3635

2014-10-13T18:52:18+02:00

A question, when and where were the changes discussed?

Statistics: Posted by zeroAPM — 13 Oct 2014, 18:52

Re: Coming in patch 3635

2014-10-13T15:09:26+02:00

Shadows wrote:
What will be the filename for the new Mass Storage icon?
Is it going to get a new actual icon or just an optional filename reference for mods to take advantage of? I'm asking this because if it does get a new actual icon then it would create a visual bug for people with old icon mods that presume the mass storage icon is the same as the mass extractor icon.

The change that was previously implemented (It's not on github), duplicated the icon along with its name. So we would have this issue with people using old mods.

The change is only to allow ASI to do this, so if you have an idea of what to do please let me know

Statistics: Posted by Sheeo — 13 Oct 2014, 15:09

Re: Coming in patch 3635

2014-10-13T10:29:09+02:00

What will be the filename for the new Mass Storage icon?
Is it going to get a new actual icon or just an optional filename reference for mods to take advantage of? I'm asking this because if it does get a new actual icon then it would create a visual bug for people with old icon mods that presume the mass storage icon is the same as the mass extractor icon.

Statistics: Posted by Shadows — 13 Oct 2014, 10:29

Re: Coming in patch 3635

2014-10-13T03:10:03+02:00

Development work is now proceeding apace on Github, at this location: https://github.com/FAForever/fa

Statistics: Posted by IceDreamer — 13 Oct 2014, 03:10

Re: Coming in patch 3635

2014-10-13T02:56:47+02:00

Sheeo wrote:
Sir Prize wrote:Thank you devs!

I PMed Sheeo about a month ago I think asking if T1 Mass Extractors could have their strategic icon separated in the blueprints from T1 Mass Storage. It sounded like it was easy to do and would get up - any update on this?

Thanks again for your work guys.

Of course, I'll make sure to push this change later today

EDIT: It's done here: https://bitbucket.org/thepilot/forged-alliance-forever-lua/pull-request/116/new-icon-name-for-mass-storage

The link is broken.

Statistics: Posted by Shadows — 13 Oct 2014, 02:56