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

Daily Standups #27

Closed
MattPereira opened this issue Apr 18, 2024 · 10 comments
Closed

Daily Standups #27

MattPereira opened this issue Apr 18, 2024 · 10 comments
Assignees

Comments

@MattPereira
Copy link
Member

This issue is for tracking daily progress and prioritization planning

@MattPereira
Copy link
Member Author

MattPereira commented Apr 18, 2024

April 18th

Yesterday

Today

  • Communicate with Balancer about the updates
  • Offer more feedback on SDK to Balancer

@steve0xp
Copy link
Contributor

steve0xp commented Apr 18, 2024

April 18th Stand-Up

Yesterday

  • Worked on custom pool PR, added in fake ERC20s to work with in terms of creating new pool factories and pools.
  • Opening up new comments in issue 1.3b Tooling Development (Smart Contracts) #26 regarding specifics on CreatePoolFromFactoryExample.s.sol, mainly revolving around what wallet is used for deployment/initialization, and which is used for front-end interactions.

Today

  • Combine details from DeployCustomPoolFactoryExample.s.sol with CreatePoolFromFactoryExample.s.sol
  • Make actual PR, debug compilation errors
  • Update README with how things are meant to work for "off-the-shelf" use of ScaffoldBalancer tool.

Thoughts / Blockers / Questions

@steve0xp
Copy link
Contributor

steve0xp commented Apr 19, 2024

April 19th Stand-Up

Yesterday

  • Combine details from DeployCustomPoolFactoryExample.s.sol with CreatePoolFromFactoryExample.s.sol
  • Make actual PR, debug compilation errors
  • Thought through test template setup. Posting it in a new issue, but first want to clarify it to make sure questions are clear for Balancer Labs.

Today

  • Clarifying questions related to tests && error I'm hitting when running DeployCustomPoolFactoryAndNewPoolExample.s.sol
  • Adding more docs ideas to issue Docs CRs #17 and pinging BLabs on it. Might just open issues for them too on this if it is relevant.
    • Mainly about:
      • Testing with their repo as a custom pool developer (can we just run make test and have the vault carry out "Balancer scoped" tests similar to how Yearn strategies are tested against the Yearn architecture?). This would not be in the scope of this tooling project.
      • Use of custom factory for pool deployments, and aspects related to it.

Thoughts / Blockers / Questions

  • I didn't set up that meeting with John for this week, so I'll request for one for next week.

@steve0xp steve0xp pinned this issue Apr 19, 2024
@MattPereira
Copy link
Member Author

MattPereira commented Apr 19, 2024

April 19th

Yesterday

  • Communicate with Balancer about the updates in response to feedback
  • Offer more feedback on SDK to Balancer

Today

@MattPereira
Copy link
Member Author

MattPereira commented Apr 23, 2024

April 22nd

Weekend

  • no progress to report

Today

@MattPereira
Copy link
Member Author

MattPereira commented Apr 23, 2024

April 23rd

Yesterday

Today

Thoughts / Blockers / Questions

  • Would the Balancer SDK work with a pool that is only deployed to a forked network?
    • make on chain calls to mock the api call
  • What is the plan moving forward?
    • Get to milestone 1 asap
  • Handle errors on my side or SDK side?
    • Leave it to balancer for now

@steve0xp
Copy link
Contributor

steve0xp commented Apr 23, 2024

April 23rd Stand-Up

Yesterday / The Last Few Days

  • Make actual PR, debug compilation errors
  • Clarifying questions related to tests && error I'm hitting when running DeployCustomPoolFactoryAndNewPoolExample.s.sol
  • Adding more docs ideas to issue Docs CRs #17 and pinging BLabs on it. Might just open issues for them too on this if it is relevant.

Today

Thoughts / Blockers / Questions

  • From meeting with John and Matt today: I’ll provide update(s) by Thursday EoD on where we’re at with milestone 1, specifically the smart contracts as front end is almost complete. Depending on where things are at by EoW we may transition smart contracts work over to Balancer team. This is TBD based on this week's progress.

@MattPereira
Copy link
Member Author

MattPereira commented Apr 24, 2024

2024-04-24

Yesterday

  • Code along with @steve0xp teaching me how to use v3 test comprehension to write script that uses pool factory contract to handle set up of new pool

Today

Thoughts / Blockers / Questions

  • At some point, I think we should migrate back to foundry version of SE-2 instead of hardhat 🙃

@steve0xp
Copy link
Contributor

April 24th Stand-Up

Yesterday

Today

Thoughts / Blockers / Questions

  • New issues to open but not prioritize:
    • Reformat foundry scripts to hardhat/TS (if needed)
    • Upgrade script and tests to use Permit2
    • Use Balancer Test ERC20 and mimic their test setup more so as to be in line with their development process

@MattPereira
Copy link
Member Author

MattPereira commented Apr 25, 2024

2024-04-25

Yesterday

  • Started testing custom pools deployed through factory with the frontend
  • Ran into bug and communicated with John about it on TG

Today

Thoughts / Blockers / Questions

  • We are not taking advantage of SE-2 scripts with our current SE-2 Hardhat setup, either we should also write pool deployment scripts in hardhat or switch to ( back 😮‍💨 ) to Foundry version of SE-2
  • There is also a new release of SE-2 since we started this one where they upgraded from viem v1 to viem v2 which is a pretty heavy lift from my understanding

@steve0xp steve0xp unpinned this issue May 9, 2024
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

No branches or pull requests

2 participants