Skip to content

Add sigs for some other hotbar/action methods - #1925

Merged
wolfcomp merged 2 commits into
aers:mainfrom
KazWolfe:xivdeck0.5.0
Aug 19, 2026
Merged

Add sigs for some other hotbar/action methods#1925
wolfcomp merged 2 commits into
aers:mainfrom
KazWolfe:xivdeck0.5.0

Conversation

@KazWolfe

Copy link
Copy Markdown
Contributor

NOTE: Breaking Change to GetCostTextForSlot due to an incorrect return type. No usages found.

@github-actions github-actions Bot added the requested changes Changes requested through comments label Aug 18, 2026

@wolfcomp wolfcomp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For now this will have to be casted between until 8.0

Comment thread FFXIVClientStructs/FFXIV/Client/UI/Misc/RaptureHotbarModule.HotbarSlot.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/UI/Misc/RaptureHotbarModule.HotbarSlot.cs Outdated

@wolfcomp wolfcomp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Signature not at subroutine start

Comment thread FFXIVClientStructs/FFXIV/Client/UI/Misc/RaptureHotbarModule.HotbarSlot.cs Outdated
Co-authored-by: wolfcomp <4028289+wolfcomp@users.noreply.github.com>
@github-actions github-actions Bot removed the requested changes Changes requested through comments label Aug 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Breaking Changes

Member exists in left but not in right

FFXIVClientStructs.FFXIV.Client.UI.Misc.RaptureHotbarModule.HotbarSlot.Delegates: 1

GetCostTextForSlot: 2

  • uint Invoke(FFXIVClientStructs.FFXIV.Client.UI.Misc.RaptureHotbarModule.HotbarSlot*, FFXIVClientStructs.FFXIV.Client.UI.Misc.RaptureHotbarModule.HotbarSlotType, uint)
  • uint EndInvoke(System.IAsyncResult)

FFXIVClientStructs.FFXIV.Client.UI.Misc.RaptureHotbarModule: 1

HotbarSlot: 1

  • uint GetCostTextForSlot(FFXIVClientStructs.FFXIV.Client.UI.Misc.RaptureHotbarModule.HotbarSlotType, uint)

@github-actions github-actions Bot added the breaking change PR contains breaking changes and wont be merged before a new patch label Aug 18, 2026
@wolfcomp
wolfcomp merged commit f5ae314 into aers:main Aug 19, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change PR contains breaking changes and wont be merged before a new patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants