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

feat: Features.CustomStats #130

Open
wants to merge 21 commits into
base: dev
Choose a base branch
from
Open

feat: Features.CustomStats #130

wants to merge 21 commits into from

Conversation

Misfiy
Copy link
Member

@Misfiy Misfiy commented Oct 6, 2024

Description

Describe the changes
Add Features.CustomStats

What is the current behavior? (You can also link to an open issue here)

What is the new behavior? (if this is a feature change)
Ability to set MaxStamina & MaxHumeShield

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:


Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentations

Submission checklist

  • I have checked the project can be compiled
  • I have tested my changes and it worked as expected

Patches (if there are any changes related to Harmony patches)

  • I have checked no IL patching errors in the console

Other

  • Still requires more testing

@Misfiy Misfiy requested review from VladTheCow and VALERA771 October 6, 2024 12:36
@Misfiy Misfiy added enhancement New feature or request and removed Transpiler base-game-bug Bug from the main game labels Oct 6, 2024
@Misfiy Misfiy removed their assignment Oct 6, 2024
@Misfiy
Copy link
Member Author

Misfiy commented Oct 15, 2024

Thats things i think would make it able to set to 0 instead of being forced to something like 0.0001 or into the negatives which may cause issues

@github-actions github-actions bot added Transpiler base-game-bug Bug from the main game labels Oct 19, 2024
/// <summary>
/// Fix for <see cref="CustomStaminaStat"/>.
/// </summary>
[HarmonyPatch(typeof(FpcStateProcessor), nameof(FpcStateProcessor.UpdateMovementState))]
Copy link

@louis1706 louis1706 Oct 19, 2024

Choose a reason for hiding this comment

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

I am pretty sure there a way to not having to patch this event it's also in the wrong file

Copy link
Member Author

Choose a reason for hiding this comment

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

Should prob be transpiler but idk

VALERA771 pushed a commit to NotZer0Two/OLDEXILED that referenced this pull request Nov 5, 2024
@github-actions github-actions bot removed the Events label Nov 18, 2024
/// <summary>
/// Fix for <see cref="CustomStaminaStat"/>.
/// </summary>
[HarmonyPatch(typeof(FpcStateProcessor), nameof(FpcStateProcessor.UpdateMovementState))]
Copy link
Member Author

Choose a reason for hiding this comment

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

Should prob be transpiler but idk

@Misfiy
Copy link
Member Author

Misfiy commented Nov 19, 2024

This needs testing, I made it work previously but this is a redo of it so not 100% itll work

@Misfiy Misfiy mentioned this pull request Dec 3, 2024
8 tasks
@Misfiy Misfiy changed the base branch from dev to scpsl14 December 3, 2024 09:01
@Misfiy Misfiy requested a review from louis1706 December 3, 2024 14:20
Copy link

@louis1706 louis1706 left a comment

Choose a reason for hiding this comment

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

I will review later on why this is needed

EXILED/Exiled.Events/Patches/Generic/FixStaminaStat.cs Outdated Show resolved Hide resolved
@Misfiy Misfiy requested a review from louis1706 December 4, 2024 10:44
@louis1706 louis1706 changed the title Features.CustomStats feat: Features.CustomStats Dec 26, 2024
@louis1706 louis1706 changed the base branch from scpsl14 to dev December 26, 2024 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API base-game-bug Bug from the main game enhancement New feature or request Transpiler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants