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

Side missions framework #621

Merged
merged 41 commits into from
Jun 23, 2019
Merged

Side missions framework #621

merged 41 commits into from
Jun 23, 2019

Conversation

Dubjunk
Copy link
Contributor

@Dubjunk Dubjunk commented Jun 21, 2019

Q A
Bug fix? no
New feature? yes
Needs wipe? no
Fixed issues #530

Description:

This PR provides the ability to use side- and eventmissions in the Liberation.
With the provided framework you're able to register missions and control them.
Additionally there are several options to define the missions.

Content:

  • Basic side mission framework
  • Functions to add side missions dynamically
  • Provide a template for adding side missions as sqf files
  • Dialog for side missions starting etc.

Successfully tested on:

  • Local MP
  • Dedicated MP

Compatibility checked with:

@Dubjunk Dubjunk added this to the Sprint 13 milestone Jun 21, 2019
@Dubjunk Dubjunk self-assigned this Jun 21, 2019
@Dubjunk Dubjunk changed the title V0.97 s13 530 Side missions framework Jun 21, 2019
@Dubjunk Dubjunk marked this pull request as ready for review June 22, 2019 22:48
@Dubjunk Dubjunk requested review from Wyqer and veteran29 June 22, 2019 22:48
Copy link
Member

@Wyqer Wyqer left a comment

Choose a reason for hiding this comment

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

Some minor things you might consider to add in this sprint or in the next iteration.
From what I see atm it looks good and handy for the future.
I'm just missing (maybe I've overlooked it) the handling of a "spawn mission" and a file for it in the example mission. So spawning the objectives/vehicles/markers/etc.
Planned for the next iteration, I guess?

@Dubjunk Dubjunk merged commit 6049e1c into v0.97S13 Jun 23, 2019
@Dubjunk Dubjunk deleted the v0.97S13-530 branch June 23, 2019 21:20
@Dubjunk Dubjunk added done and removed review/QA labels Jun 23, 2019
@Wyqer Wyqer mentioned this pull request Jun 30, 2019
@Wyqer Wyqer mentioned this pull request Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants