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

Add drystreak forestry #5796

Open
TastyPumPum opened this issue Mar 22, 2024 · 8 comments
Open

Add drystreak forestry #5796

TastyPumPum opened this issue Mar 22, 2024 · 8 comments

Comments

@TastyPumPum
Copy link
Contributor

Add forestry to the drystreak command, have it compare fox whistle, golden pheasant egg, petal garland and sturdy beehive parts against forestry events completed.

@TastyPumPum
Copy link
Contributor Author

hey @ellipsis-dev can you implement this?

Copy link

ellipsis-dev bot commented Jun 13, 2024

To have Ellipsis create a Pull Request with the implementation, comment below with "@ellipsis-dev". You can include suggestions or additional instructions. (Re-planning is not yet supported.)

Implementation Steps

Step 1: Modify the 'drystreak' command to include options for the specified items

In the file src/mahoji/commands/tools.ts, modify the 'drystreak' command to include options for the items 'fox whistle', 'golden pheasant egg', 'petal garland', and 'sturdy beehive parts'. You can do this by adding these items to the itemOption function call in the command options. Make sure to handle the case where these items are not part of the loot from a monster, as the current implementation of the command is focused on monster drops.

Step 2: Modify the 'drystreak' command to handle forestry events

In the file src/mahoji/commands/tools.ts, extend the dryStreakEntities array to include a new entity for forestry events. This new entity should have a custom query to fetch the dry streak data for forestry events, similar to the existing entities for monster drops. Also, ensure that the command can handle the item drops from forestry events, which might be different from the drops handled by the existing entities. You can do this by incorporating checks and logic similar to those used for monster drops, adjusting for the specific data structure and functionalities of forestry events.

Step 3: Add tests for the new functionality

Extend the existing tests in tests/integration/dryStreak.test.ts by adding new test cases that simulate forestry events and check the outcomes against expected results. Use the implementation in src/tasks/minions/woodcuttingActivity.ts as a reference for how forestry events are handled and what outcomes (like item drops) to expect. Handle any dependencies or setup required for these tests.


Questions? Check out our documentation.

@TastyPumPum
Copy link
Contributor Author

@ellipsis-dev can you make the pr for this? However step 1 isn't required.

Copy link

ellipsis-dev bot commented Jun 13, 2024

To have Ellipsis create a Pull Request with the implementation, comment below with "@ellipsis-dev". You can include suggestions or additional instructions. (Re-planning is not yet supported.)

Implementation Steps

Step 1: Modify the 'drystreak' command to handle forestry events

In the file src/mahoji/commands/tools.ts, extend the dryStreakEntities array to include a new entity for forestry events. This new entity should have a custom query to fetch the dry streak data for forestry events, similar to the existing entities for monster drops. Also, ensure that the command can handle the item drops from forestry events, which might be different from the drops handled by the existing entities. You can do this by incorporating checks and logic similar to those used for monster drops, adjusting for the specific data structure and functionalities of forestry events.

Step 2: Add tests for the new functionality

Extend the existing tests in tests/integration/dryStreak.test.ts by adding new test cases that simulate forestry events and check the outcomes against expected results. Use the implementation in src/tasks/minions/woodcuttingActivity.ts as a reference for how forestry events are handled and what outcomes (like item drops) to expect. Handle any dependencies or setup required for these tests.


Questions? Check out our documentation.

@TastyPumPum
Copy link
Contributor Author

@ellipsis-dev can you make the pr for the above?

Copy link

ellipsis-dev bot commented Jun 13, 2024

To have Ellipsis create a Pull Request with the implementation, comment below with "@ellipsis-dev". You can include suggestions or additional instructions. (Re-planning is not yet supported.)

Implementation Steps

Step 1: Modify the 'drystreak' command to handle forestry events

In the file src/mahoji/commands/tools.ts, extend the dryStreakEntities array to include a new entity for forestry events. This new entity should have a custom query to fetch the dry streak data for forestry events, similar to the existing entities for monster drops. Also, ensure that the command can handle the item drops from forestry events, which might be different from the drops handled by the existing entities. You can do this by incorporating checks and logic similar to those used for monster drops, adjusting for the specific data structure and functionalities of forestry events.

Step 2: Add tests for the new functionality

Extend the existing tests in tests/integration/dryStreak.test.ts by adding new test cases that simulate forestry events and check the outcomes against expected results. Use the implementation in src/tasks/minions/woodcuttingActivity.ts as a reference for how forestry events are handled and what outcomes (like item drops) to expect. Handle any dependencies or setup required for these tests.


Questions? Check out our documentation.

@TastyPumPum
Copy link
Contributor Author

@ellipsis-dev

Copy link

ellipsis-dev bot commented Jun 13, 2024

To have Ellipsis create a Pull Request with the implementation, comment below with "@ellipsis-dev". You can include suggestions or additional instructions. (Re-planning is not yet supported.)

Implementation Steps

Step 1: Modify the 'drystreak' command to handle forestry events

In the file src/mahoji/commands/tools.ts, extend the dryStreakEntities array to include a new entity for forestry events. This new entity should have a custom query to fetch the dry streak data for forestry events, similar to the existing entities for monster drops. Also, ensure that the command can handle the item drops from forestry events, which might be different from the drops handled by the existing entities. You can do this by incorporating checks and logic similar to those used for monster drops, adjusting for the specific data structure and functionalities of forestry events.

Step 2: Add tests for the new functionality

Extend the existing tests in tests/integration/dryStreak.test.ts by adding new test cases that simulate forestry events and check the outcomes against expected results. Use the implementation in src/tasks/minions/woodcuttingActivity.ts as a reference for how forestry events are handled and what outcomes (like item drops) to expect. Handle any dependencies or setup required for these tests.


Questions? Check out our documentation.

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

No branches or pull requests

1 participant