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

Added support for the Character Soulbinds API #165

Merged
merged 1 commit into from
Nov 25, 2020

Conversation

danjagnow
Copy link
Collaborator

Added support for the Character Soulbinds API. Added an ICharacterSoulbindsApi interface to describe the capabilities of the API. The IWarcraftClient interface implements this interface via the IProfileApi interface. Resolves #162.

Added support for the Character Soulbinds API.  Added an ICharacterSoulbindsApi interface to describe the capabilities of the API.  The IWarcraftClient interface implements this interface via the IProfileApi interface.
@danjagnow danjagnow added this to the 5.0 milestone Nov 24, 2020
@danjagnow danjagnow self-assigned this Nov 24, 2020
@danjagnow
Copy link
Collaborator Author

@willwolfram18, this change unfortunately introduces several types (ConduitReference, CovenantReference, SoulbindReference, and TechTalentReference) that are likely to merge badly with work in progress for #159. I'll be happy to defer the merge and resolve any conflicts if that helps.

Copy link
Collaborator

@willwolfram18 willwolfram18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No worries on the overlapping type references. I have to resolve some merge conflicts on the Resources.resx already 😅 I don't mind resolving some object types as well. I'm going to try get some tests in place to validate I've got all the attributes on the types i'm working with and then I'll work on getting the merges fixed in my branch.

@danjagnow danjagnow merged commit b79e57b into master Nov 25, 2020
@danjagnow danjagnow deleted the character-soulbinds-api branch November 25, 2020 13:51
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.

Character Soulbinds API
2 participants