Skip to content

Add Stop Alarm button to Alarm Bot - #146

Open
bukulu-am wants to merge 1 commit into
king04aman:mainfrom
bukulu-am:add-stop-alarm-button
Open

Add Stop Alarm button to Alarm Bot#146
bukulu-am wants to merge 1 commit into
king04aman:mainfrom
bukulu-am:add-stop-alarm-button

Conversation

@bukulu-am

Copy link
Copy Markdown

Summary

This PR adds a "Stop Alarm" button to the Alarm Bot, allowing users to stop the ringing sound immediately. Previously, users had to close the entire application to stop the alarm. It also adds a seconds dropdown for more precise time setting.

Changes Made

  • Added alarm_active global variable to control alarm state
  • Created stop_alarm() function to stop the sound
  • Added "Stop Alarm" button to the UI (red background for visibility)

Testing

  • Set alarm for 1 minute ahead -> alarm rings -> Stop works
  • Sound stops immediately when clicking Stop
  • No errors on application close

Screenshot

(none)

Live Project Link

N/A - This is a desktop application

Developer Checklist

  • Adhered to the guidelines outlined in the README & Contributing file.
  • Maintained the correct directory structure (e.g., ProjectName/...yourfiles).
  • Please ensure to include a README file for enhanced project clarity. (Alarm Bot already has README)
  • Starred the repository (optional - done)

@github-actions

Copy link
Copy Markdown
Contributor

👋 @bukulu-am 👋

We're delighted to have your pull request! Please take a moment to check our contributing guidelines and ensure you've filled out the PR template for a smooth process. We will review it soon.

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.

1 participant