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

Verify Extension: [847] Battlefield 1 Vortex Extension #15510

Closed
15 tasks
insomnious opened this issue Apr 10, 2024 · 5 comments
Closed
15 tasks

Verify Extension: [847] Battlefield 1 Vortex Extension #15510

insomnious opened this issue Apr 10, 2024 · 5 comments

Comments

@insomnious
Copy link
Contributor

insomnious commented Apr 10, 2024

Verify Extension: Battlefield 1 Vortex Extension

Use this template to verify game extensions for Vortex ​

Extension Information

Get all relevant metadata and links from nexusmods.com

Extension Name Battlefield 1 Vortex Extension
Extension Mod ID 847
Extension URL https://www.nexusmods.com/site/mods/847
Game Name Battlefielld 1
Game Domain battlefield1
Game URL https://www.nexusmods.com/battlefield1

Check to see if there is an existing extension for this game

Existing Extension URL

Checked with

Extension Version
Vortex Version

Instructions

TBC

Verification checklist

  • If there is an existing extension for this game, is this functionally better?
  • Is the extension named correctly?
  • Is it packaged correctly?
  • Is artwork correct?
  • Is the changelog accurate?
  • Does it install into Vortex?
  • Does it correctly discover the game?
  • Does it successfully install a mod?
  • Does it successfully install a collection?
  • Does the game run correctly with the mods installed? ​

Adding extension to manifest

​ When complete, the verified extension needs adding to our manifest.

  • GitHub Action run
  • Manifest file manually checked for errors
  • Contacted author
  • Asked Community to enable Vortex for the game
  • Discord #vortex-announcements updated.
@IDCs
Copy link
Contributor

IDCs commented Apr 16, 2024

The extension is not in a working state.

To unblock:

  • The image is cropped at the bottom. Please use a 640 x 360 px image for the gameart.
    Image

  • The findGame function is set to execute before any of the game store extensions are loaded and therefore Vortex is unable to load your extension.

// This:
function makeFindGame(api, gameSpec) {
    return util.GameStoreHelper.findByAppId(gameSpec.discovery.ids)
      .then((game) => game.gamePath);
}

// Should be this:
function makeFindGame(api, gameSpec) {
    return () => util.GameStoreHelper.findByAppId(gameSpec.discovery.ids)
      .then((game) => game.gamePath);
}

@ChemGuy1611
Copy link

Thanks for pointing that out. Just a heads up that I grabbed that code block from the github documentation, so that should probably be corrected over there.

I did not have any issue loading this extension on my machine., but I have added the arrow function as shown above.

@IDCs
Copy link
Contributor

IDCs commented Apr 16, 2024

@ChemGuy1611 can you point us towards that documentation?

The up to date documentation did not take game wizard extension code into consideration.

@ChemGuy1611
Copy link

ChemGuy1611 commented Apr 16, 2024

@ChemGuy1611 can you point us towards that documentation?

The up to date documentation did not take game wizard extension code into consideration.

Nexus-Mods/vortex-api#31

In the "Game Detection" section, you'll see the findGame functions do not have the arrow function there. The Add Game Wizard does in fact use the arrow function correctly, that is why only this extension has this problem, as I did not change this function for the other extensions I've uploaded.

@ChemGuy1611
Copy link

The requested updates have been incorporated and uploaded to Nexus. Thanks again for your assistance in ensuring these extensions work well for all users!

@IDCs IDCs closed this as completed Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants