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

Ultimate Generation Setting Missing #305

Closed
BickerKFR opened this issue Feb 23, 2021 · 3 comments
Closed

Ultimate Generation Setting Missing #305

BickerKFR opened this issue Feb 23, 2021 · 3 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists lobby settings Missing or incorrect lobby settings

Comments

@BickerKFR
Copy link

I get an error importing my map on the ultimate generation lobby settings saying they're invalid properties:

		General
		{
			Ultimate Generation: 130%
			Ultimate Generation - Combat: 130%
			Ultimate Generation - Passive: 130%
@ItsDeltin ItsDeltin added lobby settings Missing or incorrect lobby settings bug Something isn't working labels Feb 23, 2021
@ItsDeltin
Copy link
Owner

Same issue as #239, wrap the general hero settings in a second 'General' object like so:

"General": {
  "General": {
    "Ultimate Generation": 125,
    "Ultimate Generation - Combat": 125,
    "Ultimate Generation - Passive": 126
  }
}

@ItsDeltin ItsDeltin added the duplicate This issue or pull request already exists label Feb 28, 2021
@BickerKFR
Copy link
Author

BickerKFR commented Mar 1, 2021

I was trying to use the import function from exactly what the current OW is dumping, it's already wrapped by "heroes". Here's a fuller example if you wanted to try and import it.

settings
{

	lobby
	{
		Allow Players Who Are In Queue: Yes
		Map Rotation: After A Game
		Max Spectators: 12
		Return To Lobby: Never
	}

	heroes
	{
		General
		{
			Ultimate Generation: 10%
			Ultimate Generation - Combat: 0%
			Ultimate Generation - Passive: 0%

			Mercy
			{
				Weapons Enabled: Caduceus Blaster Only
			}
		}
	}

and then after using the command "import from clipboard" the customGameSettings.lobby it creates has in it:

  "Heroes": {
    "General": {
      "Ultimate Generation": 10.0,
      "Ultimate Generation - Combat": 0.0,
      "Ultimate Generation - Passive": 0.0,
      "Mercy": {
        "Weapons Enabled": "Caduceus Blaster Only"
      }
    }
  },

but these Ultimate properties are yellow underlined stating "Property X is not allowed"

Hopefully that is a clearer report now.

@ItsDeltin
Copy link
Owner

There is a bug where the general hero settings are not placed in it's own section.
That should look like this:

  "Heroes": {
    "General": {
      "General": {
        "Ultimate Generation": 10.0,
        "Ultimate Generation - Combat": 0.0,
        "Ultimate Generation - Passive": 0.0
      },
      "Mercy": {
        "Weapons Enabled": "Caduceus Blaster Only"
      }
    }
  },

The first "General" object affects all teams. The second "General" section are the hero settings that affect all heroes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists lobby settings Missing or incorrect lobby settings
Projects
None yet
Development

No branches or pull requests

3 participants