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: Add RemainingDecontaminationTime property #106

Merged
merged 9 commits into from
Dec 26, 2024

Conversation

hayabusa255
Copy link

Description

Describe the changes
Added a new static property RemainingDecontaminationTime to calculate the remaining time for the decontamination process.

What is the current behavior? (You can also link to an open issue here)
Currently, there is no easy way to get the remaining time for the decontamination process.

What is the new behavior? (if this is a feature change)
The new RemainingDecontaminationTime property provides an easy way to get the remaining time for the decontamination process.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No, this PR does not introduce any breaking changes. It adds a new property without modifying existing functionality.

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

@louis1706 louis1706 added Events and removed API labels Sep 25, 2024
@hayabusa255 hayabusa255 removed their assignment Sep 28, 2024
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.

Is this the real time or the time in game ?

(15sec or 12.5 sec at the start of decontamination)

@hayabusa255
Copy link
Author

This property provides the actual remaining decontamination time from the start to the end of the game.

@louis1706
Copy link

This property provides the actual remaining decontamination time from the start to the end of the game.

still my question the real time on world time

or the time in game (that go a lot faster than IRL

@hayabusa255
Copy link
Author

The in-game time progresses faster than real-world time. If the actual decontamination time in the game is set for 15 minutes, it will be displayed as 11 minutes in-game to reflect this faster progression accurately.

This property convert to decontamination time to actual real time

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.

doesn't this exact same value ?

EXILED/Exiled.API/Features/Map.cs Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the Events label Nov 18, 2024
@VALERA771
Copy link

Pls implement yamato's suggestion, edit it if you want

@louis1706 louis1706 changed the title Add RemainingDecontaminationTime property feat: Add RemainingDecontaminationTime property Dec 26, 2024
@louis1706 louis1706 merged commit 92d7f68 into ExMod-Team:dev Dec 26, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants