This repository was archived by the owner on Nov 18, 2021. It is now read-only.
Fix problem with queue freezing#412
Merged
ZeusJunior merged 1 commit intodevelopmentfrom Apr 4, 2020
Merged
Conversation
Closed
idinium96
approved these changes
Apr 4, 2020
viggjona
approved these changes
Apr 4, 2020
ZeusJunior
approved these changes
Apr 4, 2020
|
Please approve |
Hhanuska
approved these changes
Apr 4, 2020
Nicklason
added a commit
that referenced
this pull request
Apr 4, 2020
* src: fix problem with detecting killstreak Fixes: #386 * src: fix problem with checking escrow Fixes: #374 * add recorded trades time from first trade made in !trades command (#382) * add recorded trades time from first trade made * bring the trade records to the top Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: fix trades command * src: pluralize item names * Add admins to friendsToKeep (#407) * fix typo and add missing variables found in schema (#408) Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: use getAdmins function for friendsToKeep * src: use amount that the bot can trade when pluralizing * Add option to clear pricelist (#411) * src: add all option to remove command * change confirm setting * src: fix problem checking errors when sending offer * update modules * src: fix problem with queue freezing (#412) Fixes #396 Co-authored-by: IdiNium <47635037+idinium96@users.noreply.github.com> Co-authored-by: idinium96 <idinium96@users.noreply.github.com> Co-authored-by: Mohammad Abdullah <49647622+mabdu11ah@users.noreply.github.com>
Nicklason
added a commit
that referenced
this pull request
Apr 11, 2020
* src: fix problem with detecting killstreak Fixes: #386 * src: fix problem with checking escrow Fixes: #374 * add recorded trades time from first trade made in !trades command (#382) * add recorded trades time from first trade made * bring the trade records to the top Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: fix trades command * src: pluralize item names * Add admins to friendsToKeep (#407) * fix typo and add missing variables found in schema (#408) Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: use getAdmins function for friendsToKeep * src: use amount that the bot can trade when pluralizing * Add option to clear pricelist (#411) * src: add all option to remove command * change confirm setting * src: fix problem checking errors when sending offer * update modules * src: fix problem with queue freezing (#412) Fixes #396 * src: add message command (#395) * src: add message command * follow eslint * check if messages are enabled for user too * check once Co-authored-by: menoob003 <44436740+menoob003@users.noreply.github.com> * change a few things about the message command * fix problem with enabling messages * rename env var * add env var to templates * i am a goof * i am a goof 2 * i am a goof 3 * i am a goof 4 * i am goof 5? * it is now very nice Co-authored-by: IdiNium <47635037+idinium96@users.noreply.github.com> Co-authored-by: idinium96 <idinium96@users.noreply.github.com> Co-authored-by: Mohammad Abdullah <49647622+mabdu11ah@users.noreply.github.com> Co-authored-by: Zeckatz <44436740+Zeckatz@users.noreply.github.com> Co-authored-by: menoob003 <44436740+menoob003@users.noreply.github.com>
Nicklason
added a commit
that referenced
this pull request
Apr 12, 2020
* src: fix problem with detecting killstreak Fixes: #386 * src: fix problem with checking escrow Fixes: #374 * add recorded trades time from first trade made in !trades command (#382) * add recorded trades time from first trade made * bring the trade records to the top Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: fix trades command * src: pluralize item names * Add admins to friendsToKeep (#407) * fix typo and add missing variables found in schema (#408) Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: use getAdmins function for friendsToKeep * src: use amount that the bot can trade when pluralizing * Add option to clear pricelist (#411) * src: add all option to remove command * change confirm setting * src: fix problem checking errors when sending offer * update modules * src: fix problem with queue freezing (#412) Fixes #396 * src: add message command (#395) * src: add message command * follow eslint * check if messages are enabled for user too * check once Co-authored-by: menoob003 <44436740+menoob003@users.noreply.github.com> * change a few things about the message command * fix problem with enabling messages * rename env var * add env var to templates * i am a goof * i am a goof 2 * i am a goof 3 * i am a goof 4 * i am goof 5? * it is now very nice * Review offers (#361) * src: add command checks * src: implement trades command for manual review * src: change description for trades command * src: remove unused argument * src: add functionality for review of offers If review is enabled then the bot won't decline - invalid items - overstocked - invalid value * src: manually accept / decline offers * forgot to uncomment admin check * src: fix offer logging * src: notify about offers that are pending for review * src: fix problem with no action from onNewTradeOffer handler * src: always notify reviews to admins * src: always decline escrow * src: add metadata to offer action * add missing arguments * src: find all offer reasons * add missing join * fix problem with polldata compatibility * check for escrow before marking the offer for review * add error message to accept / decline trade commands * add missing argument * fix problem with not applying action to offer * i am a goof * always message admins when offer is pending for review Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * always display offer values in metal (#415) * disable reviews by default Co-authored-by: IdiNium <47635037+idinium96@users.noreply.github.com> Co-authored-by: idinium96 <idinium96@users.noreply.github.com> Co-authored-by: Mohammad Abdullah <49647622+mabdu11ah@users.noreply.github.com> Co-authored-by: Zeckatz <44436740+Zeckatz@users.noreply.github.com> Co-authored-by: menoob003 <44436740+menoob003@users.noreply.github.com>
Nicklason
added a commit
that referenced
this pull request
May 7, 2021
* src: fix problem with detecting killstreak Fixes: #386 * src: fix problem with checking escrow Fixes: #374 * add recorded trades time from first trade made in !trades command (#382) * add recorded trades time from first trade made * bring the trade records to the top Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: fix trades command * src: pluralize item names * Add admins to friendsToKeep (#407) * fix typo and add missing variables found in schema (#408) Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: use getAdmins function for friendsToKeep * src: use amount that the bot can trade when pluralizing * Add option to clear pricelist (#411) * src: add all option to remove command * change confirm setting * src: fix problem checking errors when sending offer * update modules * src: fix problem with queue freezing (#412) Fixes #396 Co-authored-by: IdiNium <47635037+idinium96@users.noreply.github.com> Co-authored-by: idinium96 <idinium96@users.noreply.github.com> Co-authored-by: Mohammad Abdullah <49647622+mabdu11ah@users.noreply.github.com>
Nicklason
added a commit
that referenced
this pull request
May 7, 2021
* src: fix problem with detecting killstreak Fixes: #386 * src: fix problem with checking escrow Fixes: #374 * add recorded trades time from first trade made in !trades command (#382) * add recorded trades time from first trade made * bring the trade records to the top Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: fix trades command * src: pluralize item names * Add admins to friendsToKeep (#407) * fix typo and add missing variables found in schema (#408) Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: use getAdmins function for friendsToKeep * src: use amount that the bot can trade when pluralizing * Add option to clear pricelist (#411) * src: add all option to remove command * change confirm setting * src: fix problem checking errors when sending offer * update modules * src: fix problem with queue freezing (#412) Fixes #396 * src: add message command (#395) * src: add message command * follow eslint * check if messages are enabled for user too * check once Co-authored-by: menoob003 <44436740+menoob003@users.noreply.github.com> * change a few things about the message command * fix problem with enabling messages * rename env var * add env var to templates * i am a goof * i am a goof 2 * i am a goof 3 * i am a goof 4 * i am goof 5? * it is now very nice Co-authored-by: IdiNium <47635037+idinium96@users.noreply.github.com> Co-authored-by: idinium96 <idinium96@users.noreply.github.com> Co-authored-by: Mohammad Abdullah <49647622+mabdu11ah@users.noreply.github.com> Co-authored-by: Zeckatz <44436740+Zeckatz@users.noreply.github.com> Co-authored-by: menoob003 <44436740+menoob003@users.noreply.github.com>
Nicklason
added a commit
that referenced
this pull request
May 7, 2021
* src: fix problem with detecting killstreak Fixes: #386 * src: fix problem with checking escrow Fixes: #374 * add recorded trades time from first trade made in !trades command (#382) * add recorded trades time from first trade made * bring the trade records to the top Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: fix trades command * src: pluralize item names * Add admins to friendsToKeep (#407) * fix typo and add missing variables found in schema (#408) Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * src: use getAdmins function for friendsToKeep * src: use amount that the bot can trade when pluralizing * Add option to clear pricelist (#411) * src: add all option to remove command * change confirm setting * src: fix problem checking errors when sending offer * update modules * src: fix problem with queue freezing (#412) Fixes #396 * src: add message command (#395) * src: add message command * follow eslint * check if messages are enabled for user too * check once Co-authored-by: menoob003 <44436740+menoob003@users.noreply.github.com> * change a few things about the message command * fix problem with enabling messages * rename env var * add env var to templates * i am a goof * i am a goof 2 * i am a goof 3 * i am a goof 4 * i am goof 5? * it is now very nice * Review offers (#361) * src: add command checks * src: implement trades command for manual review * src: change description for trades command * src: remove unused argument * src: add functionality for review of offers If review is enabled then the bot won't decline - invalid items - overstocked - invalid value * src: manually accept / decline offers * forgot to uncomment admin check * src: fix offer logging * src: notify about offers that are pending for review * src: fix problem with no action from onNewTradeOffer handler * src: always notify reviews to admins * src: always decline escrow * src: add metadata to offer action * add missing arguments * src: find all offer reasons * add missing join * fix problem with polldata compatibility * check for escrow before marking the offer for review * add error message to accept / decline trade commands * add missing argument * fix problem with not applying action to offer * i am a goof * always message admins when offer is pending for review Co-authored-by: idinium96 <idinium96@users.noreply.github.com> * always display offer values in metal (#415) * disable reviews by default Co-authored-by: IdiNium <47635037+idinium96@users.noreply.github.com> Co-authored-by: idinium96 <idinium96@users.noreply.github.com> Co-authored-by: Mohammad Abdullah <49647622+mabdu11ah@users.noreply.github.com> Co-authored-by: Zeckatz <44436740+Zeckatz@users.noreply.github.com> Co-authored-by: menoob003 <44436740+menoob003@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #396