Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

Update exchange.js #340

Closed
wants to merge 1 commit into from
Closed

Update exchange.js #340

wants to merge 1 commit into from

Conversation

kuhugo
Copy link
Contributor

@kuhugo kuhugo commented Jun 27, 2017

buy / sell orders will cancel themselves after 1 day.

@kuhugo
Update exchange.js
5d94c40 
buy / sell orders will cancel themselves after 1 day.
DeviaVir
DeviaVir suggested changes Jun 30, 2017
View reviewed changes
Copy link
Owner

@DeviaVir DeviaVir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would expect this is an optional parameter that you can define yourself, not as a static thing set for everyone.

@txssseal
Copy link

+1 to config

@DeviaVir DeviaVir mentioned this pull request Oct 6, 2017
Merged
2 tasks
@DeviaVir
Copy link
Owner

DeviaVir commented Oct 6, 2017

Supserseded by #596

@DeviaVir DeviaVir closed this Oct 6, 2017
@rfxn rfxn mentioned this pull request Nov 4, 2022
Open
@RafalSladek RafalSladek mentioned this pull request Nov 5, 2022
Open
@tkhieu tkhieu mentioned this pull request Nov 5, 2022
Open
@rfxn rfxn mentioned this pull request Dec 25, 2022
Open
@tkhieu tkhieu mentioned this pull request Dec 26, 2022
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@DeviaVir DeviaVir DeviaVir requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kuhugo @txssseal @DeviaVir