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

ODHack: Integrate Ekubo TWAMM to Treasury #84

Open
tensojka opened this issue May 22, 2024 · 9 comments
Open

ODHack: Integrate Ekubo TWAMM to Treasury #84

tensojka opened this issue May 22, 2024 · 9 comments
Assignees

Comments

@tensojka
Copy link
Contributor

Extend treasury.cairo to support setting up TWAMM orders on Ekubo. This is a complex task.

Definition of Done

The Cairo code works as expected, has nearly full test coverage, is reasonably documented (docstrings or similar suffice) and readable.

@od-hunter
Copy link

@tensojka can you assign this project to me please?

@princeibs
Copy link
Contributor

Can I hop on this task?

@ghost
Copy link

ghost commented May 23, 2024

@tensojka im ready to start work, can I hop on this?

@tensojka
Copy link
Contributor Author

@od-hunter @princeibs

Please read CONTRIBUTING.md and outline how you would approach the task and provide an estimated delivery timeline, as it's a complex task

@od-hunter
Copy link

My estimated delivery timeline is 3 days, starting from when I’m assigned the project. I’m a full stack developer with very wide knowledge on this and to tackle this complex task, I will take the following approach:

  1. Understand the Existing Codebase. That is I’ll review and study(how it handled TWAMM), I’ll also check existing documentation.
  2. I’ll Research TWAMM Orders, to make sure I have a solid understanding of what TWAMM orders are. I’ll then determine the essential components and processes required to set up TWAMM orders.
  3. Plan the Extension, break Down the Task (divide the task into smaller, manageable sub-tasks).
  4. I’ll implement the Code, testing each part thoroughly. I’ll follow best practices for readability, including proper naming conventions, modular functions, and in-code comments where necessary. I’ll also add docstrings or similar documentation to explain the purpose and functionality of your code.
  5. I’ll ensure almost full test coverage by writing unit tests for all new and modified functions.
  6. Provide clear instructions on how to use the new TWAMM order functionality in treasury.cairo.

Please I’d love to be given this opportunity.

@tensojka
Copy link
Contributor Author

@od-hunter Can you open a draft PR so I can see the status and progress?

@od-hunter
Copy link

Hello @tensojka I am still making some more research as this task is complex and I wouldn’t want to make mistakes. I will send the PR today, but if I’m not able to, I’ll send it tomorrow unfailingly please. But I promise you I'm on it, Thank you sir.🙏🏾

@od-hunter
Copy link

@tensojka while making my research, I found this interface for Ekubo TWAMM: https://github.com/EkuboProtocol/abis/blob/main/src/extensions/interfaces/twamm.cairo, which I believe is what I am supposed to integrate with Treasury contract.

But I couldn't find the implementation of the interface (I guess it's not opensource). In this case, do you mind adding more details about what part of the interface I'm supposed to extend?

@tensojka
Copy link
Contributor Author

@od-hunter You're not supposed to extend the interface, you should add an ability for the treasury to start DCA orders on the TWAMM. Yes, it's not opensource, if you have questions, I would suggest you ask in the Ekubo Discord

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants