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

🦾 catch error like a boss #547

Merged
merged 2 commits into from
Apr 4, 2021
Merged

Conversation

idinium96
Copy link
Member

@idinium96 idinium96 commented Apr 4, 2021

  • βœ… catch error on updateOldPrices method so it will not get caught on setupPricelist method (which will result in using temporary key prices).

image

@idinium96 idinium96 merged commit 936a023 into development Apr 4, 2021
@idinium96 idinium96 mentioned this pull request Apr 4, 2021
idinium96 added a commit that referenced this pull request Apr 6, 2021
#548

## Improvements/Major Changes
- 🩹 improvement on PPU - closes issue #506 (read #520)
    - once updated to v3.8.0, your pricelist.json is no longer usable on the lower versions.
    - only effective on newly bought items.

## Added
- ✨ an option to show detailed time taken (#549)
    - new options: `tradeSummary.showDetailedTimeTaken` (default is `true`).

## Changed
- πŸ”¨ include the old group name in high-value disabled item notification (#544)

## Fixes
- πŸ”¨ fixed bot crash if enableSocket set to false (#543)
- πŸ› possible fix negative profit when accepting invalid items (#545, a344d0d)
- πŸ”¨ fixed catch the wrong error while updating `oldPrices` (mostly if use custom pricer) (#547)
@idinium96 idinium96 deleted the refactor/setupPricelist branch April 7, 2021 01:50
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