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

[WIP] New mods repository #416

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

ArseniyShestakov
Copy link
Member

@ArseniyShestakov ArseniyShestakov commented Feb 21, 2018

More information on repository is available on wiki:
https://wiki.vcmi.eu/Mods_repository

Changes:

  • Mods game started with now serialized inside StartInfo. So list of mods available in save file header.
  • New "Show mods" button for loading screen. Text changes to orange "Mods error!" when some mods missing or have different version.
  • Player will also get warning on attempt to start game while mods check did not pass.

TODO:

  • Lib and Launcher: store commit hash for mods in addition to version.
  • Client: implement VLC reload when gameplay is finished.
  • Launcher: Implement command-line interface for mod management.
  • Client: feature to run launcher in background to get missing mods and then reload VLC.
  • Launcher: Implement redirection support for Qt 5.5 and older.
  • Mods repository: implement checking of repository.json against schema on Travis.

IDEAS:

  • Another nice advantage of git. Any mod might have own "repository.json" inside repo so player will be able to add this single mod to his launcher even if mod not included in main VCMI repository, or if only older version is included.

@ArseniyShestakov ArseniyShestakov changed the title [WIP] Feature/new mods repository [WIP] New mods repository Feb 21, 2018
@nullkiller
Copy link
Member

I expect this work is no longer actual

@nullkiller nullkiller marked this pull request as draft September 26, 2022 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants