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 permit2 subSteps #1141

Open
wants to merge 7 commits into
base: feat/addLiquidityPermit2
Choose a base branch
from
Open

Conversation

agualis
Copy link
Contributor

@agualis agualis commented Oct 2, 2024

agualis and others added 5 commits October 2, 2024 11:37
* chore: migrate to dRPC

* chore: disable foundry cache in GHA
* chore: Init

* feat: add prop slippage count down animation

* refactor: remove logs and comments

* feat: remove animation

* chore: Adjust balances

* chore: Adjust balances in provider

* chore: Cleanup

* chore: Revert numbers change

---------

Co-authored-by: Anastasios <[email protected]>
Copy link

vercel bot commented Oct 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
frontend-v3 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 2, 2024 2:28pm
test-v3 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 2, 2024 2:28pm

@@ -23,9 +23,9 @@ export function Steps({ transactionSteps }: Props) {
colorMode={colorMode}
isLastStep={isLastStep(index)}
/>
{!isLastStep(index) && (
{/* {!isLastStep(index) && (
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@uiuxxx I commented out this vertical separator that only worked when no subSteps are present. We need something that looks good in both scenarios

function SubSteps({ color, subSteps }: { color: string; subSteps: SubSteps }) {
return (
<Box mt="1" pl="2" p="1">
{subSteps.gas === 0 && (
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@uiuxxx For now I only show Signature: Free for permit substeps. I want to finish implementing the pool actions and then we can talk about the gas implementation depending of the available time left.

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