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

[Keyboard Brightness] Error toast appears even when toggle is successful #11966

Closed
clasick opened this issue Apr 24, 2024 · 4 comments
Closed
Labels
bug Something isn't working extension: keyboard-brightness Issues related to the keyboard-brightness extension extension Issues related to one of the extensions in the Store

Comments

@clasick
Copy link

clasick commented Apr 24, 2024

Extension

https://www.raycast.com/huzef44/keyboard-brightness

Description

When using the "Toggle Keyboard Brightness" action, the error toast "Failed Toggling Keyboard Brightness" appears even when the toggle is successful, i.e., the keyboard lights turn on/off as desired.

image

Steps To Reproduce

  1. Install the Keyboard Brightness extension
  2. Open Raycast
  3. Find and run the Toggle Keyboard Brightness command
  4. Observe the physical keyboard lights are toggled from on to off, or off to on
  5. Observe error toast appear in the bottom middle of the screen: "Failed Toggling Keyboard Brightness"

Current Behaviour

Error toast appears when toggle is successful

Expected Behaviour

Error toast should only be shown during toggle failure

@clasick clasick added bug Something isn't working extension Issues related to one of the extensions in the Store labels Apr 24, 2024
@raycastbot raycastbot added the extension: keyboard-brightness Issues related to the keyboard-brightness extension label Apr 24, 2024
@raycastbot
Copy link
Collaborator

raycastbot commented Apr 24, 2024

Thank you for opening this issue!

🔔 @neo773 @belyakov-am @DannySeidel @pernielsentikaer you might want to have a look.

💡 Author and Contributors commands

The author and contributors of huzef44/keyboard-brightness can trigger bot actions by commenting:

  • @raycastbot close this issue Closes the issue.
  • @raycastbot rename this issue to "Awesome new title" Renames the issue.
  • @raycastbot reopen this issue Reopen the issue.

@clasick clasick changed the title [Keyboard Brightness] Error toast appears even when toggle is success [Keyboard Brightness] Error toast appears even when toggle is successful Apr 24, 2024
@DannySeidel
Copy link
Contributor

I tried out different commits to see when the error first appeared and it seems that the error was introduced in commit 33d4d2d by @pernielsentikaer in PR #11740. With the previous commits the error isn't triggered.

@pernielsentikaer what was the purpose of the changes in the commit? Because the Menubar extension and the alerts seem to be in sync.

@pernielsentikaer
Copy link
Collaborator

I added a fix here for it @DannySeidel

@pernielsentikaer
Copy link
Collaborator

Should be fixed now, thanks for flagging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working extension: keyboard-brightness Issues related to the keyboard-brightness extension extension Issues related to one of the extensions in the Store
Projects
None yet
Development

No branches or pull requests

4 participants