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

Re-enable expeditions (basic functionality) #123

Merged
merged 27 commits into from
Aug 1, 2020

Conversation

mdziekon
Copy link
Owner

@mdziekon mdziekon commented Aug 1, 2020

Summary:

The old expeditions code needed a complete rewrite due to its incompatibility with "new" Flights handling systems. For now, it only implements two events: nothing happened and resources found. More events should be added in future Pull Requests.

Changelog:

  • Complete rewrite of the expeditions code.
  • Expeditions (as a mission) has been re-enabled by default.
  • Added feature flags capability, allowing to disable expeditions by changing .FEATURES__EXPEDITIONS__ISENABLED constant's value from true to false.
  • Fixed expedition shortcut in Galaxy view to always point to the currently selected galaxy & solar system's coords
    • Previously, it would stay at the "initial" coordinates
  • Fleet destruction reason codes are easier to use in code now (unique Enum)

Is migration required?

YES

Related issues or PRs:

(fleet row updates, development log insertion & fleet archive insertion)
@mdziekon mdziekon added pr:bugfix All pull requests related to fixing bugs pr:enhancement_request All pull requests related to enhancements proposals labels Aug 1, 2020
@mdziekon mdziekon added this to In progress in Basic modernisation via automation Aug 1, 2020
@mdziekon mdziekon merged commit cbd64cf into master Aug 1, 2020
Basic modernisation automation moved this from In progress to Done Aug 1, 2020
@mdziekon mdziekon deleted the mdziekon/90/refactor-expeditions-part01 branch August 1, 2020 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:bugfix All pull requests related to fixing bugs pr:enhancement_request All pull requests related to enhancements proposals
Projects
Development

Successfully merging this pull request may close these issues.

Reinstate expedition fleet mission's code
1 participant