New Secondary Objective 'Humanitarian Aid'#894
New Secondary Objective 'Humanitarian Aid'#894UselessFodder wants to merge 5 commits intoKillahPotatoes:masterfrom
Conversation
This mission costs 5 intel by default, selects a random friendly town, and requests the players bring 2 supply crates to within 50m of the town center. Once the supplies are in place, it deletes those supply crates and rewards the player with +5% to civilian reputation (default). The intel cost and % chance in civilian reputation are configurable in the kp_liberation_config. The mission will also detect if there are currently no friendly towns, display a notification, and refund the correct intel cost.
|
This is my first pull request, so I'm sorry if the formatting is off. Let me know what you guys think! |
|
Just doing a ping to ask if anyone has had to check this commit out? I think it would be a good inclusion to the base game. |
|
@UselessFodder I will check them out as soon as I figure out how to get this PR to show up on my fork. |
|
Works great, nice job, integrated into our live fork. |
ColinM9991
left a comment
There was a problem hiding this comment.
Just pulled this mission and found an issue with the English localisation. Other than this, nice work.
|
Thanks for catching that!
…On Fri, Jun 17, 2022 at 11:04 AM ColinM9991 ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Just pulled this mission and found an issue with the English localisation.
Other than this, nice work.
------------------------------
In Missionframework/stringtable.xml
<#894 (comment)>
:
> + <Key ID="STR_SECONDARY_MISSION3">
+ <Original>Humanitarian Aid</Original>
+ <French>Recherche et Sauvetage</French>
+ <German>Helikopterbergung</German>
+ <Spanish>Búsqueda y rescate</Spanish>
+ <Russian>Найти и Спасти</Russian>
+ <Italian>Cerca e Soccorri</Italian>
+ <Chinesesimp>搜索与营救</Chinesesimp>
+ <Chinese>搜索與營救</Chinese>
+ <Turkish>Ara ve kurtar</Turkish>
+ <Portuguese>Busca e Resgate</Portuguese>
+ <Korean>수색 및 구출</Korean>
+ <Czech>Záchranná mise</Czech>
+ </Key>
+ <Key ID="STR_SECONDARY_BRIEFING3">
+ <Original><t size='1.3' color='#ffa000'>HUMANITARIAN AID</t><br/><br/><t size='1'>A nearby settlement has requested supplies, head out and deliver them to the highlighted settlement.</t><br/><br/><t size='1'>This will help win hearts and minds in the local region and ultimately help limit any locals engaging us in open and guerilla combat.</t><br/><br/><t size='1.15' color='#00a0ff'>Cost: 5<img image='\A3\Ui_f\data\GUI\Cfg\Ranks\general_gs.paa'/></t><br/><t size='1.15' color='#ffa000'>Reward: 5% increase in Civil Reputation.</t><br/><br/><t size='10'><img image='res\secondary\fob_obj.jpg/></t></Original>
There's an apostrophe missing at the very end for the English localisation.
It should read
image='res\secondary\fob_obj.jpg'/></t>
Currently it is
image='res\secondary\fob_obj.jpg/></t>
—
Reply to this email directly, view it on GitHub
<#894 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABKO3XFDG4FJKMCVPQRCDXDVPSVZFANCNFSM5D7BHIWQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Thanks!
UselessFodder
Twitch <https://mail.google.com/mail/u/0/://twitch.tv/UselessFodder> -
YouTube <https://youtube.com/UselessFodder>
Twitter <https://twitter.com/UselessFodder> - Instagram
<https://instagram.com/UselessFodder>
Discord <http://discord.uselessfodder.com/> - Website
<http://uselessfodder.com/>
|
Added a ' to line 7378 in stringtable.xml as per @ColinM9991's suggestion.
|
Just one little thing, you have Otherwise this is a great idea for a side mission <3 |
|
Thanks for the feedback! I'm not sure if there's a way to make that entry
dynamic, but I will change it to just not say any percentage until I can
figure it out
…On Thu, Feb 23, 2023 at 7:26 PM NZF-JDWang ***@***.***> wrote:
Just one little thing, you have KP_liberation_civ_supplies_impact in the
config which allows you to set the cvi rep gain, however the stringtable is
hard coded to 5%, so if you change KP_liberation_civ_supplies_impact to
say 20, the sidemission will still say 5%
Otherwise this is a great idea for a side mission <3
—
Reply to this email directly, view it on GitHub
<#894 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABKO3XDJRWG7EZGWFN5QRW3WZAL6DANCNFSM5D7BHIWQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Thanks!
UselessFodder
Twitch <https://mail.google.com/mail/u/0/://twitch.tv/UselessFodder> -
YouTube <https://youtube.com/UselessFodder>
Twitter <https://twitter.com/UselessFodder> - Instagram
<https://instagram.com/UselessFodder>
Discord <http://discord.uselessfodder.com/> - Website
<http://uselessfodder.com/>
|
You could use format by popping a |
|
Oh, wow, that makes a lot of sense. I'll test that out when I get a chance.
On Fri, Feb 24, 2023 at 10:13 AM ColinM9991 ***@***.***>
wrote:
… Thanks for the feedback! I'm not sure if there's a way to make that entry
dynamic, but I will change it to just not say any percentage until I can
figure it out
… <#m_-7046176233429153805_>
On Thu, Feb 23, 2023 at 7:26 PM NZF-JDWang *@*.*> wrote: Just one little
thing, you have KP_liberation_civ_supplies_impact in the config which
allows you to set the cvi rep gain, however the stringtable is hard coded
to 5%, so if you change KP_liberation_civ_supplies_impact to say 20, the
sidemission will still say 5% Otherwise this is a great idea for a side
mission <3 — Reply to this email directly, view it on GitHub <#894
(comment)
<#894 (comment)>>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKO3XDJRWG7EZGWFN5QRW3WZAL6DANCNFSM5D7BHIWQ
<https://github.com/notifications/unsubscribe-auth/ABKO3XDJRWG7EZGWFN5QRW3WZAL6DANCNFSM5D7BHIWQ>
. You are receiving this because you were mentioned.Message ID: @.*>
-- Thanks! UselessFodder Twitch
https://mail.google.com/mail/u/0/://twitch.tv/UselessFodder - YouTube
https://youtube.com/UselessFodder Twitter
https://twitter.com/UselessFodder - Instagram
https://instagram.com/UselessFodder Discord
http://discord.uselessfodder.com/ - Website http://uselessfodder.com/
You could use format <https://community.bistudio.com/wiki/format> by
popping a %1 in the stringtable text.
—
Reply to this email directly, view it on GitHub
<#894 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABKO3XDFPQ5CR4AIYCCOULTWZDT43ANCNFSM5D7BHIWQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Thanks!
UselessFodder
Twitch <https://mail.google.com/mail/u/0/://twitch.tv/UselessFodder> -
YouTube <https://youtube.com/UselessFodder>
Twitter <https://twitter.com/UselessFodder> - Instagram
<https://instagram.com/UselessFodder>
Discord <http://discord.uselessfodder.com/> - Website
<http://uselessfodder.com/>
|
So, I spent some time looking at this recently and I'm not sure how to utilize format in this situation as the text goes through a localization and a parseText to display the values on the secondary mission UI. Here is the route: This code calls for the _briefings text: This pulls the text from stringtable.xml here: This is the same format as the original secondary mission such as FOB_Hunt: I can probably change the Secondary UI display for all of the missions to be in formatted text and pull the configuration values for each mission, but for now I think I am going to say that's a much bigger change than a simple secondary mission addition and leave the text to be the same format as the rest of the default missions. @ColinM9991 @NZF-JDWang |
|
How can I download it? |
Description:
Content:
Successfully tested on: