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

891: Fix LastLook expired transactions #895

Merged
7 commits merged into from
Apr 17, 2024
Merged

891: Fix LastLook expired transactions #895

7 commits merged into from
Apr 17, 2024

Conversation

ghost
Copy link

@ghost ghost commented Apr 10, 2024

Fixes #891

Expiry of LastLook transactions are now handled in the transactionHooks file.

Removed all logic for keeping track of expired transactions from store. This is considered bad practice, you should only store data in your store. Adding timeouts will create side effects and will make cleanup more complicated.

Added a SubmittedOrderUnderConsideration interface for LastLook orders that are submitted but not have a transaction hash yet and are sent for consideration to LastLook.

Removed the last-look and request-for-quote types. All orders are now the same, except when it has a isLastLook=true. Then the hash can be optional.

@ghost ghost requested review from dmosites, gpxl-dev and mikestarrdev April 10, 2024 20:43
@ghost ghost changed the base branch from develop to feature/869-rebased April 10, 2024 20:43
@ghost ghost changed the title Feature/891 891: Fix LastLook expired transactions Apr 10, 2024
Base automatically changed from feature/869-rebased to develop April 17, 2024 20:32
@ghost ghost force-pushed the feature/891 branch from 6a18ae7 to 9fe827a Compare April 17, 2024 20:43
@ghost ghost merged commit fc81ada into develop Apr 17, 2024
2 checks passed
@ghost ghost deleted the feature/891 branch April 17, 2024 20:44
This pull request was closed.
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.

Fix LastLook expired transactions
1 participant