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

Feature + Backend: Pest Trap Features #3303

Open
wants to merge 27 commits into
base: beta
Choose a base branch
from

Conversation

DavidArthurCole
Copy link
Contributor

What

Adds option to get beeped/titled/chatted when Pest Traps are full or out of bait, and adds Pest Trap and its related Tab Widgets to TabWidgetDisplay.
Created a new PestTrapApi, as this took quite a bit of code. Also cleaned up some GardenApi.isInGarden() instances, but those are gonna get pulled out to a new PR anyways.

Images

image

Changelog New Features

  • Added warnings for Pest Traps being full or out of bait. - Daveed
  • Added Pest Trap tab widgets to Tab Widget Display. - Daveed

Changelog Technical Details

  • Added PestTrapApi. - Daveed

@github-actions github-actions bot added the Backend A backend pull request that will be merged soon label Jan 25, 2025
@hannibal002 hannibal002 added this to the 2.0.0 milestone Jan 25, 2025
Copy link

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 25, 2025
# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/features/garden/visitor/VisitorListener.kt
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 25, 2025
Copy link

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 25, 2025
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 25, 2025
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 25, 2025
Copy link

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot added Detekt Has detekt problem and removed Merge Conflicts There are open merge conflicts with the beta branch. labels Jan 26, 2025
Copy link

One or more Detekt Failures were detected:

  • DicerRngDropTracker.kt#L3: Imports must be ordered in lexicographic order without any empty lines in-between with "java", "javax", "kotlin" and aliases in the end. This should then be followed by pre-processed imports.

@github-actions github-actions bot removed the Detekt Has detekt problem label Jan 26, 2025
@github-actions github-actions bot added the Detekt Has detekt problem label Jan 26, 2025
Copy link

One or more Detekt Failures were detected:

@github-actions github-actions bot removed the Detekt Has detekt problem label Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend A backend pull request that will be merged soon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants