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

EliteAPI fix, NetworkCompatiblity fix and more #200

Merged
merged 11 commits into from
Aug 21, 2020
Merged

EliteAPI fix, NetworkCompatiblity fix and more #200

merged 11 commits into from
Aug 21, 2020

Conversation

xiaoxiao921
Copy link
Member

  • Fix an oob array exception with EliteAPI that was adding incorrectly custom elites of tier 2 and above.
  • Better display of missing mods for end users when a mod mismatch happens
  • Better log message if the monomod patch of r2api is missing
  • R2API now adds itself to the networked mod list for syncing purposes when NetworkingAPI is active (the message classes from that API could be different across r2api versions and cause problems for example)
  • Fix a rare bug ( computer specific ? ) where the NetworkCompatiblityHandler wasnt correctly retrieving the arguments of the NetworkCompatibility attribute
  • Better detection of duplicate assemblies that could be sitting in the plugins folder of an end user

@Reinms
Copy link
Contributor

Reinms commented Aug 21, 2020

Perhaps it would be a good to have an error logged if a mod is requesting both networkAPI to be loaded, and requesting to be clientside. Those two things combined could cause weird mismatch cases. (and don't make any sense anyway)

@tristanmcpherson tristanmcpherson merged commit 8c8475b into risk-of-thunder:master Aug 21, 2020
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