Skip to content

Update Client.Game.UI.Achievement - #1890

Merged
wolfcomp merged 12 commits into
aers:mainfrom
AtmoOmen:Achievement
Aug 6, 2026
Merged

Update Client.Game.UI.Achievement#1890
wolfcomp merged 12 commits into
aers:mainfrom
AtmoOmen:Achievement

Conversation

@AtmoOmen

Copy link
Copy Markdown
Contributor

No description provided.

@AtmoOmen

Copy link
Copy Markdown
Contributor Author

actions/checkout@v4 now blocks checking out fork PR code from pull_request_target, and both workflows use that pattern.

see https://github.com/actions/checkout/releases/tag/v4.4.0 and https://github.blog/changelog/2026-06-18-safer-pull_request_target-defaults-for-github-actions-checkout/

@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.

Rebase against latest main.

@github-actions github-actions Bot added breaking change PR contains breaking changes and wont be merged before a new patch and removed breaking change PR contains breaking changes and wont be merged before a new patch labels Jul 21, 2026
@AtmoOmen

Copy link
Copy Markdown
Contributor Author

done

@wolfcomp wolfcomp added the outdated New patches have been released since this pr and it could be wrong label Jul 28, 2026
@Haselnussbomber Haselnussbomber removed the outdated New patches have been released since this pr and it could be wrong label Jul 28, 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.

A few things I have noted down.

Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
AtmoOmen and others added 7 commits August 6, 2026 12:31
Co-authored-by: wolfcomp <4028289+wolfcomp@users.noreply.github.com>
Co-authored-by: wolfcomp <4028289+wolfcomp@users.noreply.github.com>
Co-authored-by: wolfcomp <4028289+wolfcomp@users.noreply.github.com>
@AtmoOmen

AtmoOmen commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

thanks for reviews, all resloved now

@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.

Some more stuff to be fixed up.

Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/UI/Achievement.cs
@AtmoOmen

AtmoOmen commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for such detailed reviews again. All resolved now.

@wolfcomp
wolfcomp merged commit 1c7f0d5 into aers:main Aug 6, 2026
5 checks passed
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