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

Return nullable on XivChatType.GetDetails #2143

Merged

Conversation

Haselnussbomber
Copy link
Contributor

Some XivChatType do not have a XivChatTypeInfoAttribute on them, so XivChatType.GetDetails() should return a nullable.

@Haselnussbomber Haselnussbomber requested a review from a team as a code owner December 11, 2024 06:55
@Haselnussbomber
Copy link
Contributor Author

Haselnussbomber commented Dec 11, 2024

Well, the Check API Compatibility workflow succeeded, so I guess that means it's not a breaking change?!
Renaming the PR then.

@Haselnussbomber Haselnussbomber changed the title [Api12] Return nullable on XivChatType.GetDetails Return nullable on XivChatType.GetDetails Dec 11, 2024
@KazWolfe KazWolfe enabled auto-merge (squash) December 17, 2024 15:44
@KazWolfe KazWolfe merged commit 3ccd93a into goatcorp:master Dec 17, 2024
3 checks passed
@Haselnussbomber Haselnussbomber deleted the nullable-xivchattype-getdetails branch December 17, 2024 15:58
KazWolfe added a commit that referenced this pull request Dec 17, 2024
* Use MiniGame BGM scene for About tab

* Use span comparison in GameGui.IsInLobby

* [master] Update ClientStructs (#2126)

Co-authored-by: github-actions[bot] <noreply@github.com>

* [master] Update ClientStructs (#2134)

Co-authored-by: github-actions[bot] <noreply@github.com>

* [master] Update ClientStructs (#2135)

Co-authored-by: github-actions[bot] <noreply@github.com>

* Fix config option summaries (#2138)

* Fix config option summaries

* Obsolete LockonDefaultZoom_179 with error

* Don't allow nameplate GameObject lookup when ObjectId is invalid (#2137)

* build: 11.0.2.0

* Update ClientStructs (#2139)

Co-authored-by: github-actions[bot] <noreply@github.com>

* chore: remove [Experimental] from SeStringRenderer (#2144)

It hasn't caused problem for a while (though performance probably can be
better), and looks good enough to remove this attribute.

* Update ClientStructs (#2140)

Co-authored-by: github-actions[bot] <noreply@github.com>

* Return nullable on XivChatType.GetDetails (#2143)

---------

Co-authored-by: Haselnussbomber <mail@haselnussbomber.de>
Co-authored-by: github-actions[bot] <noreply@github.com>
Co-authored-by: nebel <9887+nebel@users.noreply.github.com>
Co-authored-by: goat <goatsdev@protonmail.com>
Co-authored-by: goat <16760685+goaaats@users.noreply.github.com>
Co-authored-by: srkizer <soreepeong@gmail.com>
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