-
Notifications
You must be signed in to change notification settings - Fork 213
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
UIData Refactoring #1964
UIData Refactoring #1964
Conversation
- Add shim layer for `UIData.get` - Update types
- Ad `UIData.newGet`
[frontend] [Mon Apr 15 14:15:25 UTC 2024] - Deployed 85767fc to https://genshin-optimizer-prs.github.io/pr/1964/frontend (Takes 3-5 minutes after this completes to be available) [frontend] [Mon Apr 15 17:10:05 UTC 2024] - Deployed 885763e to https://genshin-optimizer-prs.github.io/pr/1964/frontend (Takes 3-5 minutes after this completes to be available) [frontend] [Tue Apr 16 04:30:08 UTC 2024] - Deployed b8f4bc3 to https://genshin-optimizer-prs.github.io/pr/1964/frontend (Takes 3-5 minutes after this completes to be available) [frontend] [Tue Apr 16 15:14:06 UTC 2024] - Deployed 9d11a4b to https://genshin-optimizer-prs.github.io/pr/1964/frontend (Takes 3-5 minutes after this completes to be available) [Tue Apr 16 17:46:37 UTC 2024] - Deleted deployment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/shrug
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Describe your changes
mimi
branch for mini-migrationAdded
UIData.newGet
andnewGetNodeDisplay
to replaceUIData.get
andcomputeNodeDisplay
UIData.newGet
returnsCalcResult
, then pass that value intonewGetNodeDisplay
to getCalcDisplay
.CalcResult
andCalcDisplay
should contain all information thatNodeDisplay
has, albeit in different order/format (CheckcomputeNodeDisplay
for the conversion)TODO:
UIData.getAll
,UIData.getTeamBuff
,UIData.getDisplay
to use the new type format (NodeDisplay -> CalcDisplay)UIData.get
Issue or discord link
Testing/validation
Before: with intention to remove dependency,
After:
Checklist before requesting a review (leave this PR as draft if any part of this list is not done.)
yarn run mini-ci
locally to validate format and lint.