This repository has been archived by the owner on Oct 18, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 19
Feature - Add second parameter for range method calls #529
Merged
Conversation
This file contains 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
updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes Feature/healthcheck json (#525) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes add json flag for healthcheck * use logger instead of console and pretty print it * added log check for healthcheck cli test Version bump/0.7.1 beta (#526) * 0.7.1-beta * broker version change 0.7.1-beta allow parameters for range call fixed variables for start finish
3286843
to
bcafc34
Compare
3 tasks
bradleystachurski
approved these changes
May 29, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dannypaz Looks good, just a few small comments on docs.
broker-daemon/models/fill.js
Outdated
@@ -425,13 +425,14 @@ class Fill { | |||
* Create a set of options that can be passed to a LevelUP `createReadStream` call | |||
* that limits the set to fills that belong to the given blockOrderId. | |||
* This works because all fills are prefixed with their blockOrderId and the Delimiter. | |||
* @param {string} blockOrderId - of of the block order to create a range for | |||
* @param {string} startId - of of the block order to start the range |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nit] "of of"
broker-daemon/models/order.js
Outdated
@@ -461,13 +461,14 @@ class Order { | |||
* Create a set of options that can be passed to a LevelUP `createReadStream` call | |||
* that limits the set to orders that belong to the given blockOrderid. | |||
* This works because all orders are prefixed with their blockOrderId and the Delimiter. | |||
* @param {string} blockOrderId - of of the block order to create a range for | |||
* @param {string} startId - of of the block order to start the range |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nit] "of of"
dannypaz
pushed a commit
that referenced
this pull request
May 31, 2019
updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes Feature/healthcheck json (#525) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes add json flag for healthcheck * use logger instead of console and pretty print it * added log check for healthcheck cli test Version bump/0.7.1 beta (#526) * 0.7.1-beta * broker version change 0.7.1-beta Feature - Add second parameter for range method calls (#529) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes Feature/healthcheck json (#525) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes add json flag for healthcheck * use logger instead of console and pretty print it * added log check for healthcheck cli test Version bump/0.7.1 beta (#526) * 0.7.1-beta * broker version change 0.7.1-beta allow parameters for range call fixed variables for start finish * rename rangeForBlockOrder to rangeForBlockOrderIds * comment change of of update pm2 to resolve axios vuln fix
dannypaz
pushed a commit
that referenced
this pull request
May 31, 2019
updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes Feature/healthcheck json (#525) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes add json flag for healthcheck * use logger instead of console and pretty print it * added log check for healthcheck cli test Version bump/0.7.1 beta (#526) * 0.7.1-beta * broker version change 0.7.1-beta Feature - Add second parameter for range method calls (#529) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes Feature/healthcheck json (#525) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes add json flag for healthcheck * use logger instead of console and pretty print it * added log check for healthcheck cli test Version bump/0.7.1 beta (#526) * 0.7.1-beta * broker version change 0.7.1-beta allow parameters for range call fixed variables for start finish * rename rangeForBlockOrder to rangeForBlockOrderIds * comment change of of update pm2 to resolve axios vuln fix
dannypaz
pushed a commit
that referenced
this pull request
May 31, 2019
updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes Feature/healthcheck json (#525) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes add json flag for healthcheck * use logger instead of console and pretty print it * added log check for healthcheck cli test Version bump/0.7.1 beta (#526) * 0.7.1-beta * broker version change 0.7.1-beta Feature - Add second parameter for range method calls (#529) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes Feature/healthcheck json (#525) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file Fix/network status slow (#524) * added amounts for insufficient funds errors updated enum to have unknown default value Fix - Error messaging for insufficient funds and filling orders (#522) * added amounts for insufficient funds errors * error logging for filling own order add logs to check time for network status added amounts for insufficient funds errors (#523) updated enum to have unknown default value add command to validate proto file and update circleci to check added run: to circleci command remove unused file s update circle config to install protoc install protoc try to update repos try to fix apt-get add protoc installation for circle ci try new circle setup for protoc added config use unzip instead of tar fix jessie for deps added protoc check for circle ci readd package command and remove apt-get install of protoc stuff all your protoc are belong to us echo bash env echo bash env hail mary use 64 instead run file remove help file fix the way we inflate all block orders add more logging for testing edits to deadline for network status and removed dev logs from active fund calc * added check for deadline params * fix test with market * change from string to number * fix deadline usage in order summary and wallet * fix tests for deadline changes add json flag for healthcheck * use logger instead of console and pretty print it * added log check for healthcheck cli test Version bump/0.7.1 beta (#526) * 0.7.1-beta * broker version change 0.7.1-beta allow parameters for range call fixed variables for start finish * rename rangeForBlockOrder to rangeForBlockOrderIds * comment change of of update pm2 to resolve axios vuln fix
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Description
This PR adds a second parameter to both
rangeForOrders
andrangeForFills
helpers on the Order/Fill models respectively. These changes allow those methods to performance ranges that are outside of a single block orderThis PR is related to the performance changes included in #527
Related PRs
List related PRs if applicable
Todos