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

ModListAPI fix, APISubmodule changes, ItemAPI fix. Code writing style cleanup. #107

Merged
merged 6 commits into from
Jan 16, 2020

Conversation

xiaoxiao921
Copy link
Member

  • Make code writing style more consistent with the R2API project.

  • Fix some access modifiers being incorrect.

  • ModListAPI
    File got renamed, to see the commit diff please check the commit directly 7497b99
    Fix null ref exception

  • ItemAPI
    Add a null check when custom player models don't have an ItemDisplayRuleSet available.

  • APISubmoduleHandler
    Add IsLoaded(nameof submodule) method to check if a submodule is loaded for mod makers.
    Add some xml docs and remove some debug lines that are kinda redundant.

- Make code writing style more consistent with the R2API project.
- Fix null ref with null SteamId
- added IsLoaded(nameof submodule) method to check if a submodule is loaded for mod makers
- added some xml docs
- Fixes some access modifiers being incorrect
@tristanmcpherson tristanmcpherson merged commit fe36d68 into risk-of-thunder:master Jan 16, 2020
@tristanmcpherson
Copy link
Member

Good stuff!

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.

2 participants