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

Mine Detector - Added public framework #10785

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

Conversation

johnb432
Copy link
Contributor

@johnb432 johnb432 commented Mar 8, 2025

When merged this pull request will:

  • Cleaned up component:
    • Removed redundant functions.
    • Formatted code.
    • Fixed up headers.
    • Optimised some code.
  • Made ace_minedetector_fnc_activateDetector, ace_minedetector_fnc_deactivateDetector, ace_minedetector_fnc_hasDetector and ace_minedetector_fnc_isDetectorEnabled API.
  • Enable custom minedetectors:
    • Custom sounds can be used.
    • These sounds can be played at defined intervals and distances.
    • Mine detector head position can be defined on a per mine detector basis.
      • Improved existing mine detectors by defining more accurate mine detector head positions.
  • Added documentation about existing events.

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

@johnb432 johnb432 added the kind/feature Release Notes: **ADDED:** label Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Release Notes: **ADDED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants