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

[XE] fix two bugs about learning new blood gifts #77533

Merged
merged 4 commits into from
Nov 2, 2024

Conversation

SariusSkelrets
Copy link
Contributor

Summary

Mods "[XE] fix two bugs about learning new blood gifts"

Purpose of change

Fix two bugs related to learning new blood gifts in XEDRA Evolved:
Being able to gain tier 5 powers without being a tier 5 vampire, fixing #77076.
The EoC not trying to give you powers of a lesser tier if you already have all the powers from the tier you rolled due to how it handles effect removal.

I also removed the outdated comments about the tier 4 and 5 effects being unobtainable, which isn't the case anymore.

Describe the solution

Modify the gift learning EoC to fix the aforementioned bugs

Describe alternatives you've considered

Not fixing bugs.

Testing

Set the character's intelligence to 100 so the EoC would always pick the highest tier.
Tried to get a power without having tier 5: I get told that I can't get tier 5 powers yet and obtain a tier 4 power instead.
Tried to get a power while having tier 5: I get a tier 5 power.
Tried to get a power while having all tier 5 powers: I get a tier 4 power.

Additional context

None yet

@github-actions github-actions bot requested a review from Maleclypse November 2, 2024 16:23
@github-actions github-actions bot added <Bugfix> This is a fix for a bug (or closes open issue) Mods Issues related to mods or modding [JSON] Changes (can be) made in JSON Mods: Xedra Evolved Anything to do with Xedra Evolved and removed <Bugfix> This is a fix for a bug (or closes open issue) labels Nov 2, 2024
SariusSkelrets and others added 3 commits November 2, 2024 12:25
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions and removed json-styled JSON lint passed, label assigned by github actions labels Nov 2, 2024
@Maleclypse Maleclypse merged commit 027a61a into CleverRaven:master Nov 2, 2024
18 of 24 checks passed
@SariusSkelrets SariusSkelrets deleted the gift_research_fixes branch November 2, 2024 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Mods: Xedra Evolved Anything to do with Xedra Evolved Mods Issues related to mods or modding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants