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

修复(?)刷新天赋的时候的逻辑谬误 #767

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ZM-J
Copy link
Contributor

@ZM-J ZM-J commented Jul 17, 2024

见注释

不太懂新的 promote 要怎么去拿到

@ZM-J
Copy link
Contributor Author

ZM-J commented Jul 17, 2024

@TimeRainStarSky 具体还得看是个什么个事儿

@TimeRainStarSky
Copy link
Collaborator

@TimeRainStarSky 具体还得看是个什么个事儿

这你也@我,我也不懂

@ZM-J
Copy link
Contributor Author

ZM-J commented Jul 18, 2024

@TimeRainStarSky 具体还得看是个什么个事儿

这你也@我,我也不懂

sorry,我还得blame一下内容,请个神

@ZM-J
Copy link
Contributor Author

ZM-J commented Jul 18, 2024

请神 @yoimiya-kokomi

在 commit 4eae5af 中加上的这个 avatar.isMaxTalent 判断,现在进到这里的时候貌似 avatar.isMaxTalent 不是最新的,譬如

  • 本地缓存有个角色 80 级为突破,天赋 888
  • 之后玩家进行突破但未升级,天赋 999
  • 此时 avatar.promote5,因此 avatar.isMaxTalent 仍为 True

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