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

allow a character override in TC Builds #2160

Merged
merged 5 commits into from
May 22, 2024
Merged

allow a character override in TC Builds #2160

merged 5 commits into from
May 22, 2024

Conversation

frzyc
Copy link
Owner

@frzyc frzyc commented May 21, 2024

Describe your changes

Add a way to override character base stats (level, constellation, talents) in TC Builds.
Add extra output fields when comparing between builds of different character base stats

Issue or discord link

Testing/validation

Condense the character editor, add a toggle to override the base character
image

Additional Stat displays to show difference in character base stats
image

Checklist before requesting a review (leave this PR as draft if any part of this list is not done.)

  • I have commented my code in hard-to understand areas.
  • I have made corresponding changes to README or wiki.
  • For front-end changes, I have updated the corresponding English translations.
  • I have run yarn run mini-ci locally to validate format and lint.
  • If I have added a new library or app, I have updated the deployment scripts to ignore changes as needed

@frzyc frzyc requested a review from nguyentvan7 May 21, 2024 04:28
Copy link
Contributor

github-actions bot commented May 21, 2024

[frontend] [Tue May 21 04:31:59 UTC 2024] - Deployed a7905c7 to https://genshin-optimizer-prs.github.io/pr/2160/frontend (Takes 3-5 minutes after this completes to be available)

[frontend] [Tue May 21 05:33:22 UTC 2024] - Deployed 3942255 to https://genshin-optimizer-prs.github.io/pr/2160/frontend (Takes 3-5 minutes after this completes to be available)

[frontend] [Tue May 21 15:43:42 UTC 2024] - Deployed 1b52d91 to https://genshin-optimizer-prs.github.io/pr/2160/frontend (Takes 3-5 minutes after this completes to be available)

[frontend] [Wed May 22 03:56:52 UTC 2024] - Deployed a05e7b7 to https://genshin-optimizer-prs.github.io/pr/2160/frontend (Takes 3-5 minutes after this completes to be available)

[frontend] [Wed May 22 05:38:59 UTC 2024] - Deployed e8c20d1 to https://genshin-optimizer-prs.github.io/pr/2160/frontend (Takes 3-5 minutes after this completes to be available)

[Wed May 22 05:53:29 UTC 2024] - Deleted deployment

libs/gi/page-team/src/CharProfileCharEditor.tsx Outdated Show resolved Hide resolved
libs/gi/ui/src/util/getDisplayUtil.tsx Show resolved Hide resolved
libs/gi/ui/src/util/getDisplayUtil.tsx Show resolved Hide resolved
@@ -17,6 +18,7 @@ export type BuildTcArtifactSlot = {
export type BuildTc = {
name: string
description: string
character?: Omit<ICharacter, 'key'>
Copy link
Collaborator

Choose a reason for hiding this comment

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

No visualization of override here
image

Copy link
Owner Author

Choose a reason for hiding this comment

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

Screenshot 2024-05-22 013424

@@ -17,6 +18,7 @@ export type BuildTcArtifactSlot = {
export type BuildTc = {
name: string
description: string
character?: Omit<ICharacter, 'key'>
Copy link
Collaborator

Choose a reason for hiding this comment

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

More minor, but also no golden numbers here to show these are overridden values
image

Copy link
Owner Author

Choose a reason for hiding this comment

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

Screenshot 2024-05-22 013432

@frzyc frzyc requested a review from nguyentvan7 May 22, 2024 05:37
@frzyc frzyc merged commit 54dc3b5 into master May 22, 2024
6 checks passed
@frzyc frzyc deleted the 2154 branch May 22, 2024 05:53
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.

Add a "character editor" to TC Build editor
2 participants