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

feat: add limit orders #96

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

feat: add limit orders #96

wants to merge 11 commits into from

Conversation

Fedor-Aftermath
Copy link
Contributor

Limit Orders

  • add fetch active/executed method
  • add cancellation method

Af Users

  • add fetch user pk method
  • add create user pk method

DCA

  • move logic to new af-users service
  • typo

Configs

  • add limit configs & addresses

src/general/providers/aftermath.ts Show resolved Hide resolved
src/packages/limit/api/limitApiCasting.ts Outdated Show resolved Hide resolved
src/packages/limit/api/limitApiCastingTypes.ts Outdated Show resolved Hide resolved
src/packages/limit/api/limitApiCastingTypes.ts Outdated Show resolved Hide resolved
src/packages/limit/limit.ts Outdated Show resolved Hide resolved
src/packages/userData/userData.ts Outdated Show resolved Hide resolved
src/packages/limit/api/limitApiCastingTypes.ts Outdated Show resolved Hide resolved
src/packages/limit/api/limitApiCasting.ts Outdated Show resolved Hide resolved
chore: added terms and cons transaction signing
chore: added constant for sign & terms key
refactor: removed terms & cons unused code
chore: updated dtos
chore: update DTO
chore: added status to overview
chore: update limit addys
chore: remove back indexer fetch
chore: handle new architecture
chore: typo
chore: update status & add error msg
chore: add batch cancellation
src/packages/userData/userData.ts Outdated Show resolved Hide resolved
src/packages/limitOrders/limitOrdersTypes.ts Outdated Show resolved Hide resolved
src/packages/limitOrders/limitOrdersTypes.ts Outdated Show resolved Hide resolved
src/packages/limitOrders/api/limitOrdersApiCastingTypes.ts Outdated Show resolved Hide resolved
return {
orderId: Helpers.addLeadingZeroesToType(fields.order_id),
user: Helpers.addLeadingZeroesToType(fields.user),
// TODO: - should i convert to string?
Copy link
Collaborator

Choose a reason for hiding this comment

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

??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

event return public key as a vector of bytes, so i left the q if should i convert it to a string value (for future usage mb) or may keep as an array.

src/packages/limitOrders/api/limitOrdersApiCasting.ts Outdated Show resolved Hide resolved
@@ -71,7 +71,7 @@ export type DcaClosedOrderEventOnChain = EventOnChain<{

export type DcaExecutedTradeEventOnChain = EventOnChain<{
Copy link
Collaborator

Choose a reason for hiding this comment

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

see below comments on moving code to fe

@Fedor-Aftermath Fedor-Aftermath force-pushed the fedor/limit branch 2 times, most recently from ed3a780 to b905b13 Compare December 30, 2024 17:52
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.

2 participants